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

Bug: Featured Product auto-scrolling is wonky #945

Open
jeffsnyder opened this issue Oct 28, 2022 · 2 comments
Open

Bug: Featured Product auto-scrolling is wonky #945

jeffsnyder opened this issue Oct 28, 2022 · 2 comments

Comments

@jeffsnyder
Copy link

When clicking on a Featured Product link, then clicking back, the browser zips down to the bottom of the page and then back up. It's a wild ride.

bandicam.2022-10-28.11-27-25-130.mp4

And in Firefox (not Chrome .. unsure of other browsers) clicking a Featured Product has unpredictable auto-scrolling results.

bandicam.2022-10-28.11-25-58-438.mp4
@federicobadini
Copy link
Contributor

federicobadini commented Oct 12, 2023

I've followed this issue since a very long time. Across the various Nextjs versions to which we upgraded this behaviour was little by little improved, but it is still far from being correct.
From my experience, Firefox is still problematic while the experience on chrome has improved.

There is an experimental flag scrollRestoration that aimed at resolving the issue but again this is not always working correctly.
Here is a branch in which it is enabled and you can see how, depending on the browser you use, it behaves differently: sometimes it doesn't even try to restore the scroll position.

The good news is that the Nextjs team claim they have resolved the issue in App router.
I would just stick a bit more with what we have and see.
If we manage to progressively migrate to app router we could have this fixed for free

@federicobadini
Copy link
Contributor

This small example built in App router for instance show no issue in either Chrome, FF or Safari

Scroll.position.mov

@federicobadini federicobadini removed their assignment Oct 19, 2023
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