[css-scroll-snap] Should negative scroll-padding be allowed? #1084
Assignees
Labels
Closed Accepted by CSSWG Resolution
Commenter Satisfied
Commenter has indicated satisfaction with the resolution / edits.
css-scroll-snap-1
Current Work
Needs Testcase (WPT)
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. |
ewilligers
pushed a commit
to ewilligers/csswg-drafts
that referenced
this issue
May 20, 2019
ewilligers
pushed a commit
to ewilligers/csswg-drafts
that referenced
this issue
May 21, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Was flipping through old comments and noted that we forgot to consider whether
scroll-snap-paddingshould 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: