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

Room scroll can get stuck with last message hidden #1770

Closed
dbkr opened this issue Jul 13, 2016 · 4 comments
Closed

Room scroll can get stuck with last message hidden #1770

dbkr opened this issue Jul 13, 2016 · 4 comments
Labels
A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@dbkr
Copy link
Member

dbkr commented Jul 13, 2016

scroll up in a room, switch away from that room, switch back to that room, scroll down to the bottom. For me in matrix core just now, this hides my last message below the status bar. Switching away from the room whilst scrolled to the bottom, then switching back again, fixes it.
screen shot 2016-07-13 at 14 49 05

@ara4n
Copy link
Member

ara4n commented Aug 1, 2016

i see this intermittently too. @richvdh don't suppose you remember what could cause this?

@ara4n ara4n added T-Defect ui/ux P2 S-Minor Impairs non-critical functionality or suitable workarounds exist labels Aug 1, 2016
@richvdh
Copy link
Member

richvdh commented Aug 1, 2016

Looks like geminiscrollbar thinks you are already scrolled all the way down; so this would imply some change that has made the content longer, with geminiscrollbar.update() not being called. In the past this has happened when things happen like images loading; but I thought we'd got all of them :/.

@lukebarnard1
Copy link
Contributor

Since swapping to Quantum, I've realised what's going on here.

The "Jump to first unread message" is causing the timeline viewport to shrink, but no one tells the ScrollPanel that things have resized (like we do when events change content).

To avoid bugs like this, I have a fix that detects resizes in event tiles and the scroll panel that should fix this and similar bugs: matrix-org/matrix-react-sdk#1662

@t3chguy
Copy link
Member

t3chguy commented May 11, 2022

This is no longer relevant as we got rid of the typing indicator reserved margin

@t3chguy t3chguy closed this as completed May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants