Skip to content

[cssom-view] scroll-behavior and collapsing content #2954

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

Open
Malvoz opened this issue Jul 26, 2018 · 2 comments
Open

[cssom-view] scroll-behavior and collapsing content #2954

Malvoz opened this issue Jul 26, 2018 · 2 comments

Comments

@Malvoz
Copy link
Contributor

Malvoz commented Jul 26, 2018

The issue I stumbled upon may be very style specific but I would assume this has been seen before only I could not find such issues reported. The issue happens when scroll-behavior: smooth is used and content collapses above the link destination.

Instead of trying to explain the issue with inaccurate terminology, please visit, navigate the links and experience the awkward content revealing in this codepen: https://codepen.io/anon/pen/ejRaLa

@tabatkins
Copy link
Member

Whatever terminology you use, saying something about the issue would be better than saying nothing and forcing everyone to infer what you're talking about. ^_^

I think your issue is that if, between the time a smooth scrolling operation is trigger and it completes, the position of the element being smooth-scrolled to changes, browsers currently don't take that into account, and still scroll to the original position. The final scroll position is thus confusing, because the element you were trying to scroll to isn't in the expected place (and if it moved up, may be totally off-screen).

Is this right?

@Malvoz
Copy link
Contributor Author

Malvoz commented Jul 26, 2018

Yes that is right @tabatkins. I agree, I should have elaborated further on the issue description. Thanks for taking your time!

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

No branches or pull requests

2 participants