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

PostStream scroll bug #2001

Closed
w-4 opened this issue Feb 14, 2020 · 6 comments · Fixed by #2160
Closed

PostStream scroll bug #2001

w-4 opened this issue Feb 14, 2020 · 6 comments · Fixed by #2160

Comments

@w-4
Copy link
Contributor

w-4 commented Feb 14, 2020

Bug Report

When refreshing the discussion-page in chrome sometimes it jumps 10 number forward or back to a different post.

image

Setting a timeout of 200ms for the scrollToNumber fixed the issue for me.

@stale

This comment has been minimized.

@stale stale bot added the stale Issues that have had over 90 days of inactivity label May 14, 2020
@stale

This comment has been minimized.

@stale stale bot closed this as completed Jun 13, 2020
@askvortsov1

This comment has been minimized.

@askvortsov1 askvortsov1 reopened this Jun 15, 2020
@stale stale bot removed the stale Issues that have had over 90 days of inactivity label Jun 15, 2020
@franzliedke franzliedke added this to the 0.1.0-beta.14 milestone Aug 9, 2020
@matteocontrini
Copy link
Contributor

matteocontrini commented Aug 28, 2020

Out of curiosity, does the fix for this issue also fix the issue that can be seen in this video, which was provided to me by a user?

(For future reference: the screen recording shows the user tapping on a mention, but after some time of loading the scrolling settles on the wrong message, about 10 messages above the correct one.)

@askvortsov1
Copy link
Sponsor Member

It should. There are still some things wonky with the post stream and scrubber, but I believe that particular issue should be fixed.

@matteocontrini
Copy link
Contributor

@askvortsov1 awesome, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants