Skip to content

fix: Prevent username text overflow in room invitation screen#6972

Open
aleksandernsilva wants to merge 2 commits intodevelopfrom
fix/invite-username-overflow
Open

fix: Prevent username text overflow in room invitation screen#6972
aleksandernsilva wants to merge 2 commits intodevelopfrom
fix/invite-username-overflow

Conversation

@aleksandernsilva
Copy link
Contributor

@aleksandernsilva aleksandernsilva commented Feb 6, 2026

Proposed changes

This PR fixes an issue causing text overflow when displaying full matrix user ID's in the invitation screen. The Chip component styles were adjusted to properly shrink the text in case the content in bigger than the screen max width.

Before
image

After
Screenshot 2026-02-06 at 15 09 44

Issue(s)

FEDCORE-4

How to test or reproduce

Screenshots

Types of changes

  • Added flex shrink of 1 to Chip's text container styles.

  • Bugfix (non-breaking change which fixes an issue)

  • Improvement (non-breaking change which improves a current function)

  • New feature (non-breaking change which adds functionality)

  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • [] I have added tests that prove my fix is effective or that my feature works (if applicable)
  • [] I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

Summary by CodeRabbit

Release Notes

  • Bug Fixes
    • Improved text container behavior in the Chip component to properly handle flex layout constraints.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • app/containers/Chip/__snapshots__/Chip.test.tsx.snap is excluded by !**/*.snap

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Walkthrough

Added flexShrink: 1 to the textContainer style in the Chip component, enabling the text container to shrink proportionally within flex layouts, addressing visual overflow issues where text would expand beyond intended boundaries.

Changes

Cohort / File(s) Summary
Chip Component Styling
app/containers/Chip/index.tsx
Added flexShrink: 1 to the textContainer style, enabling flexible text container shrinking in flex layouts to prevent content overflow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A chip so small, with text so wide,
Now shrinks with grace and proper pride,
No overflow shall spoil the view,
The textContainer knows just what to do! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly addresses the main change: preventing username text overflow in the room invitation screen by adjusting the Chip component's flexShrink property.
Linked Issues check ✅ Passed The code change (adding flexShrink: 1 to text container) directly addresses the objective to prevent Matrix user ID overflow on the invitation screen FEDCORE-4.
Out of Scope Changes check ✅ Passed The single modification to the Chip component's textContainer style is directly scoped to the visual overflow issue described in FEDCORE-4; no unrelated changes are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/invite-username-overflow

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

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

@aleksandernsilva aleksandernsilva changed the title fix: Prevent username text overflow on room invite fix: Prevent username text overflow in room invitation screen Feb 6, 2026
@aleksandernsilva
Copy link
Contributor Author

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@aleksandernsilva aleksandernsilva requested a deployment to experimental_android_build February 6, 2026 19:51 — with GitHub Actions Waiting
@aleksandernsilva aleksandernsilva marked this pull request as ready for review February 7, 2026 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant