Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
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 selected for processing (2)
WalkthroughSwipeToReply functionality enhanced with RTL layout support by switching to Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|


Goal
The swipe-to-reply gesture in the message list did not work correctly in RTL layouts — the swipe direction was not reversed, the reply icon appeared from the wrong side, and the icon arrow was not mirrored.
Implementation
SwipeToReply(MessageContainer.kt): UseabsoluteOffsetinstead ofoffsetso pixel offsets are not double-reversed by the layout-direction-awareModifier.offset. Detect RTL viaLocalLayoutDirectionand reverse the drag bounds (negative offset) and icon slide-in direction accordingly. The iconRowusesAlignment.CenterStartunconditionally since it is already layout-direction aware.stream_compose_ic_reply.xml: Addandroid:autoMirrored="true"so the reply arrow icon mirrors automatically in RTL.UI Changes
str_rtl.mp4
Testing
Summary by CodeRabbit
Release Notes