-
-
Notifications
You must be signed in to change notification settings - Fork 835
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
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.) |
It should. There are still some things wonky with the post stream and scrubber, but I believe that particular issue should be fixed. |
@askvortsov1 awesome, thank you! |
Bug Report
When refreshing the discussion-page in chrome sometimes it jumps 10 number forward or back to a different post.
Setting a timeout of 200ms for the scrollToNumber fixed the issue for me.
The text was updated successfully, but these errors were encountered: