Skip to content

fix(omnichannel): transfer comment text wrapping - #41901

Open
mmustafasenoglu wants to merge 1 commit into
RocketChat:developfrom
mmustafasenoglu:fix/omnichannel-transfer-comment-wrapping
Open

fix(omnichannel): transfer comment text wrapping#41901
mmustafasenoglu wants to merge 1 commit into
RocketChat:developfrom
mmustafasenoglu:fix/omnichannel-transfer-comment-wrapping

Conversation

@mmustafasenoglu

@mmustafasenoglu mmustafasenoglu commented Aug 22, 2026

Copy link
Copy Markdown

Fixes #26723

Problem

When transferring an omnichannel conversation to another agent with a comment, the comment text was not wrapping to new lines and was being truncated with '...' (ellipsis).

Solution

Added overflow-wrap: break-word and word-wrap: break-word styles to the MessageSystemBody component in the SystemMessage variant to allow long text to wrap properly.

Testing

  • Verified that the fix allows long transfer comments to wrap to multiple lines
  • The fix is minimal and only affects system message rendering

Review in cubic

Summary by CodeRabbit

  • Bug Fixes
    • Long system messages now wrap correctly instead of overflowing the message area.

Add overflow-wrap and word-wrap styles to MessageSystemBody to prevent
long transfer comments from being truncated with ellipsis.

Fixes RocketChat#26723
@mmustafasenoglu
mmustafasenoglu requested a review from a team as a code owner August 22, 2026 08:31
Copilot AI lite review requested due to automatic review settings August 22, 2026 08:31
@changeset-bot

changeset-bot Bot commented Aug 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 89c338c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@dionisio-bot

dionisio-bot Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is missing the 'stat: QA assured' label
  • This PR is missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@CLAassistant

CLAassistant commented Aug 22, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mmustafasenoglu mmustafasenoglu changed the title Fix omnichannel transfer comment text wrapping fix(omnichannel): transfer comment text wrapping Aug 22, 2026
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af689cf8-afe9-450b-85be-a1f918072a57

📥 Commits

Reviewing files that changed from the base of the PR and between 2a7de45 and 89c338c.

📒 Files selected for processing (1)
  • apps/meteor/client/components/message/variants/SystemMessage.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: cubic · AI code reviewer
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{ts,tsx,js}

📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)

**/*.{ts,tsx,js}: Write concise, technical TypeScript/JavaScript with accurate typing in Playwright tests
Avoid code comments in the implementation

Files:

  • apps/meteor/client/components/message/variants/SystemMessage.tsx
apps/meteor/**

📄 CodeRabbit inference engine (CLAUDE.md)

The main Rocket.Chat Meteor application resides in apps/meteor/; place its application code there rather than in other monorepo areas.

Files:

  • apps/meteor/client/components/message/variants/SystemMessage.tsx
🔇 Additional comments (1)
apps/meteor/client/components/message/variants/SystemMessage.tsx (1)

105-109: LGTM!


Walkthrough

MessageSystemBody now wraps long system message content. The change adds overflowWrap and wordWrap styles while preserving existing accessibility attributes.

Changes

System message wrapping

Layer / File(s) Summary
Apply system message wrapping styles
apps/meteor/client/components/message/variants/SystemMessage.tsx
MessageSystemBody now applies overflowWrap and wordWrap styles to long content. Accessibility attributes remain unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 89c33

This localized styling change allows long transfer comments to wrap instead of being truncated, with no actionable merge-blocking risk remaining beyond normal checks and review.

Suggested labels: type: bug

Suggested reviewers: tassoevan

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The change adds wrapping styles that address issue #26723 by displaying long omnichannel transfer comments across multiple lines.
Out of Scope Changes check ✅ Passed The changes are limited to wrapping styles in MessageSystemBody and align with issue #26723.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: fixing text wrapping for omnichannel transfer comments.

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.

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

Copy link
Copy Markdown
Contributor

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

Re-trigger cubic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Omnichannel transfer comment is not wrapped to multiple lines

3 participants