1231777 - (css-scroll-snap) [META] implement CSS Scroll Snap - Update implementation to latest CR spec.

Status: NEW

(Core :: Layout: Scrolling and Overflow, enhancement)

Description

[David Baron :dbaron:]

The CSS Working Group has agreed on a bunch of changes to the Scroll Snapping specification, to address issues originally raised by Mozilla folks and others.

We should implement these changes.

The working group resolutions are in:
https://lists.w3.org/Archives/Public/www-style/2015Nov/0266.html
https://lists.w3.org/Archives/Public/www-style/2015Nov/0305.html
https://lists.w3.org/Archives/Public/www-style/2015Nov/0306.html
https://lists.w3.org/Archives/Public/www-style/2015Dec/0048.html

These were motivated by a competing spec draft:
https://drafts.csswg.org/css-scroll-snap/
which is intended to be merged (but hasn't yet been) into the primary spec:
https://drafts.csswg.org/css-snappoints/

Comment 1

[Sebastian Zartner [:sebo]]

Meanwhile the competing spec. draft was merged. Because the CSS Scroll Snap Module has now CR status[1] and work on the changes doesn't seem to have begun yet, I took the liberty to adjust the summary accordingly.
There were a lot more changes since this bug was filed and the current list of properties is this:

scroll-snap-type
scroll-padding
scroll-snap-margin
scroll-snap-align
scroll-snap-stop
scroll-padding-top
scroll-padding-right
scroll-padding-bottom
scroll-padding-left
scroll-padding-inline-start
scroll-padding-block-start
scroll-padding-inline-end
scroll-padding-block-end
scroll-padding-block
scroll-padding-inline
scroll-snap-margin-top
scroll-snap-margin-right
scroll-snap-margin-bottom
scroll-snap-margin-left
scroll-snap-margin-block-start
scroll-snap-margin-inline-start
scroll-snap-margin-block-end
scroll-snap-margin-inline-end
scroll-snap-margin-block
scroll-snap-margin-inline

Sebastian

[1] https://lists.w3.org/Archives/Public/www-style/2016Oct/0122.html

Comment 2

[Sebastian Zartner [:sebo]]

> Summary: implement CSS Scroll Snap - Update implementation to latest CR spec. → [META] implement CSS Scroll Snap - Update implementation to latest CR spec.

"[META]" is obviously redundant, as there is already the 'meta' keyword set. What's the reason to additionally put it into the summary? (Asking unprejudicedly. I just want to get to know whether I should set it on other bugs, too.)

Sebastian

Comment 3

[Tantek Çelik]

(ironic meta discussion about meta) I think because it helps the META bugs pop-out better in list views such as searches when you're looking for e.g. *the* meta bug about a specific spec or feature, or in an area, component, or getting a feel for how much meta vs specific bugs etc. Seems to be established in older bugs so I stuck with the convention.

Comment 4

[Sebastian Zartner [:sebo]]

(In reply to Tantek Çelik from comment #3)
> (ironic meta discussion about meta)

;-)

> I think because it helps the META bugs
> pop-out better in list views such as searches when you're looking for e.g.
> *the* meta bug about a specific spec or feature, or in an area, component,
> or getting a feel for how much meta vs specific bugs etc. Seems to be
> established in older bugs so I stuck with the convention.

I see. Thank you for the clarification!

Sebastian

Comment 5

[Jen Simmons [:jensimmons]]

Chrome is shipping support for Scroll Snap in v69, this week. Safari already supports the undated spec. I expect this will put pressure on us to get our implementation updated. Are we working on it?

Comment 6

[David]

Any plans to work on fixing the scroll-snap-stop? https://bugzilla.mozilla.org/show_bug.cgi?id=1312165

It works quite well on Chromium/Safari and the whole feature of scroll snap becomes a bit useless without the stop (at least when you have more than 2 elements to scroll through).