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

When navigating back and forward from a video list, SPF randomly scrolls down position on video page (Chrome only) #385

Open
fireattack opened this issue Jan 11, 2016 · 2 comments

Comments

@fireattack
Copy link

I can repo this on Chrome but not Firefox.
Browser: Chrome Version 49.0.2612.0 dev-m (64-bit)
OS: Win 7 (64-bit)

Repo:

  1. Open one of the video list, e.g. https://www.youtube.com/channel/UC2bkHVIDjXS7sgrgjFtzOXQ/videos
  2. Open the first video on the third row, "Operation".
  3. Wait the page fully loaded, especially the comment section.
  4. Navigate Back to the list by press backspace or click the back button on the browser.
  5. Click on another video, the second video on the third row, "Analysis".

What happened: when you open the second video page, you will find your scroll position is not at top at very beginning, then it scroll back to top very briefly. But after the comment section is fully loaded, it scrolls down again for no reason.

What should happen: it shouldn't scroll at all.

If I use some way to disable SPF, the problem would be gone.

I made a video to show it better: https://youtu.be/R4bK5WU87oM

@fireattack fireattack changed the title When navigating back and forward from a video list, SPF randomly scrolls down (Chrome only) position on video page When navigating back and forward from a video list, SPF randomly scrolls down position on video page (Chrome only) Jan 11, 2016
@nicksay
Copy link
Contributor

nicksay commented Jan 15, 2016

@fireattack Thanks for the report. I can repo the bug, and we're looking into a fix. At this point, I think the bug is in other YouTube JS, not in SPF's scroll handling. Either way, I'll close this issue when it's fixed.

@fireattack
Copy link
Author

Thanks.

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

No branches or pull requests

2 participants