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

timeline doesn't scrolldown after you upload an image #17598

Open
ara4n opened this issue Jun 8, 2021 · 8 comments
Open

timeline doesn't scrolldown after you upload an image #17598

ara4n opened this issue Jun 8, 2021 · 8 comments
Labels
A-Media A-Timeline S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Jun 8, 2021

failing to reproduce, so i think it's a race (or perhaps i was slightly scrolled up when i uploaded the image?) but I could see the first ~40px of the thumbnail after uploading a pic and had to scroll down for the rest.

@ara4n
Copy link
Member Author

ara4n commented Jun 14, 2021

I just reproduced it (E2EE room, nothing special going on):

@ara4n
Copy link
Member Author

ara4n commented Jun 14, 2021

image

^ the clipped line at the bottom is actually the top of a screenshot I'd just pasted of a different Eleweb room.

Having scrolled down:

image

Meanwhile, i am sure i was already scrolled to the bottom of the screen.

@ara4n
Copy link
Member Author

ara4n commented Jun 14, 2021

...and I just reproduced it again in a different room (unencrypted this time); again with a PNG screenshot. This time the timeline ended up scrolled up way too high - about 100px before the point where I sent the msg.

a) This could be a good old fashioned classic scrolljump: #8565
b) This could be a race of some kind with the remote echo of the msg I've sent before the image?
c) Good news: it really feels quite reproduceable.

Might try to take a video.

@ara4n
Copy link
Member Author

ara4n commented Jun 14, 2021

repro.steps.mov

here's a video perfectly reproducing the problem in the previous comment. I enclose the screenshot used to reproduce...

Screenshot 2021-06-14 at 10 06 36

@ara4n
Copy link
Member Author

ara4n commented Jun 14, 2021

if this is an easy way to repro #8565 then perhaps we can finally solve that beast...

@ara4n ara4n added the S-Major Severely degrades major functionality or product features, with no satisfactory workaround label Jun 16, 2021
@ara4n
Copy link
Member Author

ara4n commented Jun 19, 2021

I'm seeing scrolljumps almost every time i send a message now on today's nightly - e.g:

scrolljump.mov

@ara4n
Copy link
Member Author

ara4n commented Mar 22, 2022

I think this is fixed currently.

@ara4n ara4n closed this as completed Mar 22, 2022
@ara4n
Copy link
Member Author

ara4n commented Mar 31, 2022

actually, no, it's still there, just less frequent. makes for a really jarring experience to upload a file, and then not have the timeline show it unless you scroll down :(

@ara4n ara4n reopened this Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Media A-Timeline S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants