Conversation
dlrobertson
left a comment
There was a problem hiding this comment.
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? |
dlrobertson
left a comment
There was a problem hiding this comment.
Thanks again for working on this!
domenic
left a comment
There was a problem hiding this comment.
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 |
scrollendevent 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 )