Bug 191865 - [css-scroll-snap] scroll snap align two value parsing is incorrect/backwards
Summary: [css-scroll-snap] scroll snap align two value parsing is incorrect/backwards
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: Safari 12
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar
Depends on:
Blocks:
 
Reported: 2018-11-20 12:58 PST by Majid Valipour
Modified: 2018-12-11 00:02 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Majid Valipour 2018-11-20 12:58:58 PST
CSSWG has resolved [1] to change the ordering of scroll-snap-align property to be in "block inline" order. WebKit parsing logic appear to do "inline, block" order.
I have changed Chromium implementation to match the specification but it is now different from Safari.
The current usage of two value is low but as it grows the interop issue problem grows.

Additional links:
- My backward compat analysis suggested the risk is low [2]
- Relevant WPT test [3].


I tested this on Safari 12.0.1.

[1] https://github.com/w3c/csswg-drafts/issues/2232
[2] https://github.com/w3c/csswg-drafts/issues/2232#issuecomment-399961633
[3] https://wpt.fyi/results/css/css-scroll-snap/snap-inline-block.html?label=stable&aligned
Comment 1 Radar WebKit Bug Importer 2018-11-29 10:35:08 PST
<rdar://problem/46346516>