feat: improve chat message bubble responsiveness and visual design#538
feat: improve chat message bubble responsiveness and visual design#538adityayadav176 wants to merge 2 commits into
Conversation
|
Warning Review limit reached
More reviews will be available in 56 minutes and 3 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughMessageBubble component styling was refined to improve visual hierarchy and responsiveness. The main bubble now features multi-line template styling with shadow and hover transitions, responsive width constraints, and directional border radius. Message images and audio players received updated sizing classes, while read-receipt styling now conditionally sets text color based on message direction. ChangesMessage Bubble Visual Enhancement
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@frontend/components/chat/MessageBubble.jsx`:
- Line 82: The onClick in the MessageBubble component currently calls
window.open(msg.image, "_blank") which leaves window.opener reachable; update
the onClick handler (the arrow function where window.open is invoked) to open
the image with a feature string including "noopener,noreferrer" (e.g. pass the
third argument to window.open) or otherwise use a safe anchor with rel="noopener
noreferrer" to prevent access to window.opener and suppress the referrer.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: aee6e634-4489-4197-83d8-891279b90a5f
📒 Files selected for processing (1)
frontend/components/chat/MessageBubble.jsx
|
H! @UTKARSHH20 Closes #512 |
|
H! @UTKARSHH20 |
Summary
Enhanced the chat message bubble UI to improve responsiveness, readability, and overall user experience across different screen sizes.
Changes Made
Increased mobile responsiveness with adaptive bubble widths:
max-w-[90%]on mobilemax-w-[85%]on small screensmax-w-[75%]on medium and larger screensImproved text rendering using
whitespace-pre-wrapto preserve line breaks.Added smoother visual appearance with:
shadow-mdhover:shadow-lgtransition-allduration-200Enhanced bubble styling with rounded corners for a more modern chat interface.
Improved readability and handling of long messages.
Benefits
Closes #512
Summary by CodeRabbit