Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Horizontal scroll in main room if text is too long #28434

Merged
merged 4 commits into from
Mar 20, 2023

Conversation

gabriellsh
Copy link
Member

Proposed changes (including videos or screenshots)

Removed inline styles from elements and substituted p tag for div

For a quick explanation, the p tag adds the multiple line breaks by default when copying text. We should revise in the parser how we deal with paragraphs and text blocks.

Issue(s)

Steps to test or reproduce

Send a big text without whitespaces. Ex:
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
Line won't break and screen will scroll horizontally.

Further comments

TC-540

@gabriellsh gabriellsh added this to the 6.1.0 milestone Mar 15, 2023
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #28434 (677fcdc) into develop (de9ec14) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28434      +/-   ##
===========================================
- Coverage    45.12%   45.08%   -0.04%     
===========================================
  Files          755      755              
  Lines        14614    14614              
  Branches      2093     2093              
===========================================
- Hits          6594     6589       -5     
- Misses        7722     7728       +6     
+ Partials       298      297       -1     
Flag Coverage Δ
e2e 45.04% <ø> (-0.04%) ⬇️

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

@gabriellsh gabriellsh requested a review from a team as a code owner March 16, 2023 13:21
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Mar 20, 2023
@ggazzo ggazzo merged commit f57f42f into develop Mar 20, 2023
@ggazzo ggazzo deleted the fix/horizontalScroll branch March 20, 2023 16:25
@casalsgh casalsgh modified the milestones: 6.1.0, 6.0.1 Mar 21, 2023
gabriellsh added a commit that referenced this pull request Mar 21, 2023
…avatarEmoji

* 'develop' of github.com:RocketChat/Rocket.Chat: (40 commits)
  refactor(promise.await): saveUserIdentity.ts, saveUser.js, relinquishRoomOwnerships.ts and deleteMessage.ts (#28493)
  refactor: Remove `FederationRoomEvents` model (#28488)
  refactor: Promise.await removal (#28481)
  refactor: move from canDeleteMessage to canDeleteMessageAsync (#28480)
  refactor: move from canAccessRoom to canAccessRoomAsync (#28477)
  refactor: convert getRoomRoles to async (#28485)
  refactor: Remove `Promise.await` calls from Slack Adapter (#28408)
  chore: make callbacks accept async (#28486)
  fix: Horizontal scroll in main room if text is too long (#28434)
  refactor: Types of Meteor methods (7/N) (#28482)
  refactor: Models feature parity (#28456)
  refactor: Types of Meteor methods (5/N) (#28461)
  refactor: Types of Meteor methods (6/N) (#28478)
  refactor: Convert rooms endpoints to TS (#28448)
  refactor: Convert groups endpoints to typescript (#28432)
  refactor: Convert channels endpoints to TS (#28469)
  refactor: Convert chat endpoints to TS (#28417)
  chore: Remove invalid `aria-description` from list menus (#28471)
  fix: videoconf stream (#28455)
  Chore(deps): Bump supercharge/mongodb-github-action from 1.7.0 to 1.9.0 (#28003)
  ...
d-gubert pushed a commit that referenced this pull request Mar 22, 2023
Co-authored-by: Hugo Costa <hugocarreiracosta@gmail.com>
@d-gubert d-gubert mentioned this pull request Mar 22, 2023
d-gubert pushed a commit that referenced this pull request Mar 23, 2023
Co-authored-by: Hugo Costa <hugocarreiracosta@gmail.com>
@d-gubert d-gubert mentioned this pull request Mar 23, 2023
@sampaiodiego sampaiodiego mentioned this pull request May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: QA tested stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants