Skip to content

fix: Thread list not scrolling to bottom when sending a message.#28346

Merged
kodiakhq[bot] merged 2 commits into
developfrom
fix/scrollThread
Mar 10, 2023
Merged

fix: Thread list not scrolling to bottom when sending a message.#28346
kodiakhq[bot] merged 2 commits into
developfrom
fix/scrollThread

Conversation

@gabriellsh

@gabriellsh gabriellsh commented Mar 9, 2023

Copy link
Copy Markdown
Member

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

When thread content exceeds the viewport's height on the web or desktop, there is no auto-scroll on posting a new message. As a user, if I post a new message in the thread, I then need to manually scroll down to see the message that I posted

Further comments

TC-534

@gabriellsh gabriellsh requested a review from a team as a code owner March 9, 2023 19:06
@codecov

codecov Bot commented Mar 9, 2023

Copy link
Copy Markdown

Codecov Report

Merging #28346 (783fab9) into develop (78ccb57) will increase coverage by 0.04%.
The diff coverage is 83.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28346      +/-   ##
===========================================
+ Coverage    45.29%   45.33%   +0.04%     
===========================================
  Files          759      759              
  Lines        14782    14793      +11     
  Branches      2083     2085       +2     
===========================================
+ Hits          6696     6707      +11     
  Misses        7793     7793              
  Partials       293      293              
Flag Coverage Δ
e2e 45.29% <83.33%> (+0.04%) ⬆️

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

@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Mar 10, 2023
@LevyCosta LevyCosta removed the stat: ready to merge PR tested and approved waiting for merge label Mar 10, 2023
@dionisio-bot dionisio-bot Bot added the stat: ready to merge PR tested and approved waiting for merge label Mar 10, 2023
@gabriellsh gabriellsh changed the title [FIX] Thread list not scrolling to bottom when sending a message. fix: Thread list not scrolling to bottom when sending a message. Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@kodiakhq kodiakhq Bot merged commit d224678 into develop Mar 10, 2023
@kodiakhq kodiakhq Bot deleted the fix/scrollThread branch March 10, 2023 20:39
gabriellsh added a commit that referenced this pull request Mar 13, 2023
…ketChat/Rocket.Chat into matrixSearchTestBranch

* 'feat/federation-public-room-search-2' of github.com:RocketChat/Rocket.Chat: (42 commits)
  refactor: Replace `_.isObject` with `isObject` (#28376)
  fix: small tweaks
  chore: Remove old css (#28140)
  fix: Thread list not scrolling to bottom when sending a message. (#28346)
  refactor: Federation refactor towards services + file names improvement + E2E tests fixes (#28326)
  i18n: Language update from LingoHub 🤖 on 2023-03-10Z (#28365)
  refactor: Improve sidebar search list hook (#28328)
  Chore: Change the PR title check to use conventional commits (#28358)
  [IMPROVE] Galician onboarding translation (#27908)
  [IMPROVE] Adding input type=password to AWS secrets fields (#28159)
  [FIX] Normal and multiline messages adding extra spaces upon copying/pasting (#28187)
  Chore: remove unused prop in register workspace modal (#28317)
  [FIX] App crashing when switching to Hindi (hi-In) language (#28285)
  [NEW] Registered Loom oEmbed (#28339)
  fix: create dm users before join the room
  tests: add unit tests + small tweaks
  fix: fix invite links button logic
  chore: fix lint
  Chore: Remove/replace some Meteor packages (#28189)
  Chore: Unused/dead code removal (#28155)
  ...
@casalsgh casalsgh added this to the 6.0.1 milestone Mar 21, 2023
@d-gubert d-gubert mentioned this pull request Mar 22, 2023
@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: 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