Fix excessive space between single-line messages

[gpunto]

Goal

Single-line messages have excessive padding between them

Implementation

It happened because the swipe-to-reply icon had a minimum size (48dp) so it caused the whole item to grow vertically

🎨 UI Changes

Before	After

Testing

Launch the app and check that the spacing vs messages is correct, both if the messages are small (single line) and in other cases

Summary by CodeRabbit

• Refactor
  • Refined the sizing behavior of the reply icon in message swipe actions.

[github-actions]

PR checklist ✅

All required conditions are satisfied:

• Title length is OK (or ignored by label).
• At least one pr: label exists.
• Sections ### Goal, ### Implementation, and ### Testing are filled.

🎉 Great job! This PR is ready for review.

[github-actions]

SDK Size Comparison 📏

SDK	Before	After	Difference	Status
stream-chat-android-client	5.25 MB	5.68 MB	0.43 MB	🟡
stream-chat-android-ui-components	10.60 MB	10.99 MB	0.39 MB	🟡
stream-chat-android-compose	12.81 MB	12.11 MB	-0.70 MB	🚀

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

Walkthrough

The pull request removes the minimumInteractiveComponentSize() modifier from a Box component in the SwipeToReplyIcon function. The icon container now applies only explicit size, shape clipping, and background modifiers, eliminating the automatic minimum interactive component sizing behavior.

Changes

Cohort / File(s)	Summary
Modifier Removal SwipeToReplyIcon.kt	Removed minimumInteractiveComponentSize() modifier import and its application to the icon container Box, leaving only explicit size(32.dp), clip(CircleShape), and background(...) modifiers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A swift little snip from the compose delight, 🐰
The icon sheds padding and grows small and tight,
No more minimum size rules the day,
Just thirty-two points in a circle's way! ✨
The reply swipes on, lean and light.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description includes Goal, Implementation, and UI Changes sections with before/after screenshots, but lacks detailed Testing instructions and is missing several required checklist items.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title directly addresses the main objective of the PR: fixing excessive space between single-line messages by removing the minimum interactive component size from the swipe-to-reply icon.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/messages-spacing

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}