Update message menu so long messages extend behind options#6248
Update message menu so long messages extend behind options#6248
Conversation
PR checklist ✅All required conditions are satisfied:
🎉 Great job! This PR is ready for review. |
|
SDK Size Comparison 📏
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (11)
📒 Files selected for processing (2)
WalkthroughThe changes adjust the layout behavior and dimensions of message-related UI components. The SelectedMessageMenu transitions from a scrollable column layout to a non-scrollable, height-wrapping layout with adjusted alignment. The MessageContainer increases its maximum width from 250.dp to 264.dp, providing additional horizontal space for message content. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip CodeRabbit can generate a title for your PR based on the changes with custom instructions.Set the |



Goal
The message menu was a scrollable column where the options were always at the bottom. Instead, the options should always show, and long messages should extend behind them.
While working on this, I also realized the max message width wasn't up to date with the designs, so I also changed that.
Implementation
Update the modifiers applied to the message in the message menu
🎨 UI Changes
Testing
Check the message menu on long messages
Summary by CodeRabbit
Bug Fixes
Style