-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
add scrollend
event details
#8433
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not positive about Firefox or Safari's implementation interest at the moment, but they've historically been positive
I'm new-ish, so I know very little how these things work, but Mozilla standards positions mozilla/standards-positions#240 seems to agree. FWIW the work for the particular change is tracked in bug 1797013.
Where is the spec that actually defines where this is fired? HTML is a good place to put the updates to Note that we need to reference the definition, so you'll likely need to add something for that, similar to e.g. line 3095 for |
spec is here https://drafts.csswg.org/cssom-view/#scrolling |
OK, cool. Please update the PR to link to the definition at https://drafts.csswg.org/cssom-view/#eventdef-document-scrollend like I noted above, similar to e.g. line 3095 for |
thx for the help y'all |
Sorry, this still isn't correct... you need to add something to the dependencies section, similar to line 3095. You can't just add |
gotcha. the reason i didnt just straight copy that LOC is because I didnt see |
cool, how's that @domenic? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks again for working on this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Waiting on implementer interest, then we can merge.
Implemented this in bug 1797013, so you can definitely add Gecko to implementers that are interested 😄 |
mdn issue added |
scrollend
event mdn/mdn#252Looks like I'll need help with the MDN issue? 🙂
I'm not positive about Firefox or Safari's implementation interest at the moment, but they've historically been positive about it. It's also now separated from overscroll events, which should help out since it's a much smaller task now.
fixes w3c/csswg-drafts#7949
/dom.html ( diff )
/indices.html ( diff )
/infrastructure.html ( diff )
/webappapis.html ( diff )