Repository navigation
Fix right sidebar text and tool call rendering - #177
Merged
Merged
Conversation
Replace overflow-hidden with break-words on text containers to allow proper word wrapping instead of clipping content. The CSS module already had proper word-breaking rules but the parent overflow-hidden was preventing them from working.
The TaskRenderer component for task management tool calls was also using overflow-hidden which clips text. Replace with break-words for proper text wrapping.
…erflow Add overflow-x-hidden wrapper around sidebar messages to prevent the Radix ScrollArea's internal display:table from expanding content beyond the viewport width. This ensures text wraps properly within the sidebar.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Contributor
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis pull request updates chat rendering components to improve text wrapping behavior by removing Changes
Possibly related PRs
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes
✨ Finishing touches
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
4 of 7 tasks
This was referenced Jan 30, 2026
2witstudios
added a commit
that referenced
this pull request
May 4, 2026
Hoist findActivePendingMemberByEmail above the existing-user check so the 409-on-pending path also fires when the email already maps to a temp user from a prior invitation. Without this, a second invite for an unaccepted email silently routed through the existing-user branch, updating role/permissions on the pending row and skipping the invitation email — flagged by Codex as P1. Sanitize the error message logged when sendPendingDriveInvitationEmail fails to remove control characters and cap length, addressing CodeQL js/log-injection alert #177 in notification-email-service.ts. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
8 of 14 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
Release Notes
✏️ Tip: You can customize this high-level summary in your review settings.