New issue
Advanced search Search tips

Issue 791663 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 497851



Sign in to add a comment

[css-scroll-snap] Update style parsing for CSS Scroll Snap

Project Member Reported by sunyunjia@chromium.org, Dec 4 2017

Issue description

The current CSS parsing for Snap Points doesn't match the spec. We need to

(1) Disallow percent values for scroll-snap-margin https://www.w3.org/TR/css-scroll-snap-1/#propdef-scroll-snap-margin

(2) Allow "none" value for snap-strictness https://www.w3.org/TR/css-scroll-snap-1/#snap-strictness
 

 
Blocking: 497851
Cc: majidvp@chromium.org
Status: Assigned (was: Untriaged)
Components: Blink>Scroll
Labels: Hotlist-ThreadedRendering
Summary: Update style parsing for CSS Scroll Snap (was: Update CSS for ScrollSnapPoints)
Status: Fixed (was: Assigned)
Summary: [css-scroll-snap] Update style parsing for CSS Scroll Snap (was: Update style parsing for CSS Scroll Snap)
For (1), We have disallowed percent values for scroll-snap-margin in
https://chromium-review.googlesource.com/c/chromium/src/+/782742

For (2), As in https://drafts.csswg.org/css-scroll-snap-1/#propdef-scroll-snap-type, "none" doesn't belong to snap-strictness, so we are not fixing this. 

Sign in to add a comment