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.

Sign up for GitHub

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

Jump to bottom

Timeline wobbles vertically when viewing a new room for the first time as the history loads. #18522

Closed
ara4n opened this issue Aug 13, 2021 · 2 comments
Closed

Timeline wobbles vertically when viewing a new room for the first time as the history loads. #18522

ara4n opened this issue Aug 13, 2021 · 2 comments
Labels
A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter X-Regression

Comments

@ara4n
Copy link
Member

ara4n commented Aug 13, 2021 •
edited
Loading

Steps to reproduce

View a room for the first time since launching the client (particularly an E2EE room, where the effect is either more noticeable because the history loads more slowly, or perhaps it only effects E2EE rooms).

What happened?

Watch the timeline carefully as the history is displayed on the room; notice that some messages sometimes 'dance' or 'wobble' up and down by 1px (or perhaps 0.5px?) as the history is added higher up the page, e.g. Anoa's one here. This is a regression in the last week or so, and makes things literally look wobbly and flakey.

pixel-shuffle.mov

What did you expect?

The timeline should be rock solid as the history backfills. It's acceptable (but not desirable) for replies and URL previews and media thumbnails to push events upwards if they load below them in the timeline (but ideally we should be reserving the right vertical space in the timeline for these things). But it's not acceptable for things loading above the content to make the content wobble around.

Operating system

macOS 10.15.7

Application version

nightly 2021-08-12

How did you install the app?

nightly
@ara4n ara4n added the T-Defect label Aug 13, 2021
@ara4n ara4n changed the title Timeline wobbles vertically during when viewing a new room for the first time as the history loads. Timeline wobbles vertically when viewing a new room for the first time as the history loads. Aug 13, 2021
@germain-gg germain-gg added A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Aug 13, 2021
@novocaine novocaine added S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Aug 19, 2021
@Palid
Copy link
Contributor

Palid commented Sep 2, 2021

Spent 15min or so scrolling and couldn't reproduce it anymore. Tried with new room and CPU slowdown, neither helped. Hopefully it got accidentally fixed with something.
@ara4n can you let me know if it still reproduces for you? Tested on develop.

@SimonBrandner SimonBrandner added the X-Needs-Info This issue is blocked awaiting information from the reporter label Sep 3, 2021
@MadLittleMods MadLittleMods mentioned this issue Sep 24, 2021
Closed
@novocaine
Copy link
Contributor

Closing as no recent report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter X-Regression
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@novocaine @germain-gg @ara4n @Palid @SimonBrandner