-
Notifications
You must be signed in to change notification settings - Fork 711
[css-scroll-snap] Should negative scroll-padding be allowed? #1084
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
Comments
Padding is already restricted to non-negative, and scroll-snap-padding's purpose is to make sure the snapped points don't get overlapped by other stuff positioned at the edges of the screen. A negative scroll-snap-padding would void that, as the scroller itself would overlap it. So 👍 |
Agreed. I suspect that if somebody actually came up with use cases, these would more likely be solved by introducing a |
Agenda+ to approve this normative change (restricting scroll-padding to non-negative values only). |
The CSS Working Group just discussed
The full IRC log of that discussion<dael> topic: Scroll snap<dael> github topic: https://github.com//issues/1084 <dael> astearns: Looking for WG approval for this change. <dael> astearns: Objections? <dael> RESOLVED: Accept the spec text proposed in https://github.com//issues/1084 |
I'm assuming we need one or more testcases for this change |
Reopening for testcase need. |
Was flipping through old comments and noted that we forgot to consider whether
scroll-snap-padding
should be restricted to non-negative values. In light of that discussion, I think we should go ahead and disallow negative values.Thoughts?
The text was updated successfully, but these errors were encountered: