RocketChat / RocketChat/Rocket.Chat
chore(message-parser): remove stale team collaboration styling from quote attachments
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 46.1k
- Forks
- 13.9k
- Avg merge
- 3d 3h
- Merged PRs (30d)
- 130
Description
Current Behavior
In apps/meteor/client/components/message/content/attachments/QuoteAttachment.tsx, there is an outstanding technical debt marker spanning lines 17-32:
// TODO: remove this team collaboration
This legacy CSS-in-JS block (quoteStyles) overrides standard Fuselage .rcx-attachment__details styling with hard-coded hover states and borders.
Expected Behavior
The stale quoteStyles block should be entirely removed, and the className prop should be dropped from <AttachmentContent>, allowing it to fall back to the standard, globally-maintained Fuselage component styling without legacy overrides.
Tasks
- Delete
quoteStylescss block fromQuoteAttachment.tsx - Remove
className={quoteStyles}from<AttachmentContent>
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in apps/meteor/client/components/message/content/attachments/QuoteAttachment.tsx and inspect the quoteStyles block and its use on AttachmentContent. Done means the legacy CSS-in-JS block and className prop are absent, leaving the attachment to use standard Fuselage styling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 78/100