Skip to content

Conversation

@dodaa08
Copy link
Contributor

@dodaa08 dodaa08 commented Dec 31, 2025

Proposed changes (including videos or screenshots)

Changes word-break: break-all to word-break: break-word in UserCard.tsx to prevent URLs and links in user bio from breaking mid-character. This makes links more readable and clickable while still respecting the 3-line clamp limit.

Before: Links break mid-character

image

After: Links wrap naturally

image

This change aligns UserCard with other text components in the codebase (InfoPanelText, AuditModalText, MessageText) which already use break-word for better readability.

Issue(s)

Closes #38035

Steps to test or reproduce

  1. Add a bio with URLs/links in your profile settings (e.g., https://github.com/username)
  2. Open any room and click on your avatar to open the UserCard popup
  3. Verify that links wrap naturally at word boundaries instead of breaking mid-character
  4. Verify that links remain readable, clickable, and the 3-line clamp still works

Summary by CodeRabbit

  • Style

    • Improved text wrapping in user card bios so long words are preserved and multi-line bios break more naturally.
  • Chores

    • Prepared a patch release entry documenting this bio-wrapping fix.

✏️ Tip: You can customize this high-level summary in your review settings.

- Prevents URLs/links in bio from breaking mid-word
- Improves readability while maintaining 3-line clamp
- Aligns with other text components (InfoPanelText, AuditModalText)
@dodaa08 dodaa08 requested a review from a team as a code owner December 31, 2025 20:13
@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Dec 31, 2025

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link

changeset-bot bot commented Dec 31, 2025

🦋 Changeset detected

Latest commit: bd8fc06

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 40 packages
Name Type
@rocket.chat/meteor Patch
@rocket.chat/core-typings Patch
@rocket.chat/rest-typings Patch
@rocket.chat/uikit-playground Patch
@rocket.chat/api-client Patch
@rocket.chat/apps Patch
@rocket.chat/core-services Patch
@rocket.chat/cron Patch
@rocket.chat/ddp-client Patch
@rocket.chat/fuselage-ui-kit Patch
@rocket.chat/gazzodown Patch
@rocket.chat/http-router Patch
@rocket.chat/livechat Patch
@rocket.chat/model-typings Patch
@rocket.chat/ui-avatar Patch
@rocket.chat/ui-client Patch
@rocket.chat/ui-contexts Patch
@rocket.chat/ui-voip Patch
@rocket.chat/web-ui-registration Patch
@rocket.chat/account-service Patch
@rocket.chat/authorization-service Patch
@rocket.chat/ddp-streamer Patch
@rocket.chat/omnichannel-transcript Patch
@rocket.chat/presence-service Patch
@rocket.chat/queue-worker Patch
@rocket.chat/abac Patch
@rocket.chat/federation-matrix Patch
@rocket.chat/license Patch
@rocket.chat/media-calls Patch
@rocket.chat/omnichannel-services Patch
@rocket.chat/pdf-worker Patch
@rocket.chat/presence Patch
rocketchat-services Patch
@rocket.chat/models Patch
@rocket.chat/network-broker Patch
@rocket.chat/omni-core-ee Patch
@rocket.chat/mock-providers Patch
@rocket.chat/ui-video-conf Patch
@rocket.chat/instance-status Patch
@rocket.chat/omni-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 31, 2025

Walkthrough

Updated the UserCard component's CSS to change text wrapping from word-break: break-all to word-break: break-word, and added a changeset declaring a patch release for the package.

Changes

Cohort / File(s) Summary
UserCard text wrapping
apps/meteor/client/components/UserCard/UserCard.tsx
Replaced word-break: break-all with word-break: break-word in clampStyle to prevent URLs and long words from breaking mid-character.
Release metadata
.changeset/fresh-hats-kneel.md
Added changeset documenting a patch release for @rocket.chat/meteor referencing the UserCard wrapping fix.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A rabbit nibbles on some code,
Softly mending words that flowed—
No more splits in links so neat,
Text now wraps with graceful feet. 🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The PR fully addresses issue #38035 by changing word-break from break-all to break-word, making links wrap naturally at boundaries while maintaining the 3-line clamp.
Out of Scope Changes check ✅ Passed All changes are directly scoped to fixing the word-break issue in UserCard.tsx and documenting it in the changeset file, with no unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately describes the main change: updating UserCard's word-break CSS from break-all to break-word to prevent mid-character word breaking in bio content.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@dougfabris dougfabris changed the title fix(UserCard): prevent links from breaking mid-character in bio fix: Prevent links from breaking mid-character in bio Jan 7, 2026
@dougfabris dougfabris changed the title fix: Prevent links from breaking mid-character in bio fix: Prevent words from breaking mid-character in bio Jan 7, 2026
@dougfabris dougfabris added the stat: QA assured Means it has been tested and approved by a company insider label Jan 7, 2026
@dougfabris dougfabris added this to the 8.1.0 milestone Jan 7, 2026
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jan 7, 2026
@dougfabris dougfabris changed the title fix: Prevent words from breaking mid-character in bio fix: Prevent words from breaking mid-character in user card's bio Jan 7, 2026
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.62%. Comparing base (d91fb46) to head (bd8fc06).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #38036      +/-   ##
===========================================
- Coverage    70.70%   70.62%   -0.08%     
===========================================
  Files         3146     3146              
  Lines       108826   108826              
  Branches     19557    19565       +8     
===========================================
- Hits         76941    76863      -78     
- Misses       29880    29952      +72     
- Partials      2005     2011       +6     
Flag Coverage Δ
e2e 60.16% <ø> (-0.06%) ⬇️
e2e-api 47.38% <ø> (-1.14%) ⬇️
unit 71.74% <100.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kodiakhq kodiakhq bot merged commit 4211011 into RocketChat:develop Jan 8, 2026
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

URLs in UserCard bio break mid-word, making links unreadable

2 participants