-
Notifications
You must be signed in to change notification settings - Fork 69
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
Snap Events #1008
Comments
cc @hiikezoe. I think generally this is fine, the integration looks very similar to scroll events. |
Yeah, the basic idea is pretty reasonable to me. Though I don't quite understand why SnapEvent can have only one snapTargetBlock and snapTargetInline on each axis respectively. I suppose it should be a List of nodes. Anyways it should be discussed in CSSWG not here. |
This was in fact was discussed in the CSSWG in w3c/csswg-drafts#9622 The TLDR is that when following an element after a change in layout browsers must have chosen some element to follow per https://drafts.csswg.org/css-scroll-snap-1/#re-snap so we have standarized in that issue how to choose which element is chosen (when multiple are aligned) and are exposing that chosen element in the event. |
Thanks for the pointer! I haven't followed the spec issue yet indeed. :) |
Request for Mozilla Position on an Emerging Web Specification
Snapchanging: https://github.com/argyleink/ScrollSnapExplainers/tree/main/js-snapChanging
The text was updated successfully, but these errors were encountered: