KEMBAR78
add `scrollend` event details by argyleink · Pull Request #8433 · whatwg/html · GitHub
Skip to content

Conversation

@argyleink
Copy link

@argyleink argyleink commented Oct 25, 2022


Looks 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 )

Copy link
Member

@dlrobertson dlrobertson left a 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.

@domenic
Copy link
Member

domenic commented Oct 26, 2022

Where is the spec that actually defines where this is fired? HTML is a good place to put the updates to GlobalEventHandlers, but that comes after actually defining and specifying the event.

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 event-slotchange.

@argyleink
Copy link
Author

@domenic
Copy link
Member

domenic commented Oct 27, 2022

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 event-slotchange.

@argyleink
Copy link
Author

thx for the help y'all

@domenic
Copy link
Member

domenic commented Oct 28, 2022

Sorry, this still isn't correct... you need to add something to the dependencies section, similar to line 3095. You can't just add data-x-href in the table. I really suggest copying slotchange pretty exactly, if you can.

@argyleink
Copy link
Author

Sorry, this still isn't correct... you need to add something to the dependencies section, similar to line 3095. You can't just add data-x-href in the table. I really suggest copying slotchange pretty exactly, if you can.

gotcha. the reason i didnt just straight copy that LOC is because I didnt see scroll defined anywhere like that. i'll update to add a scrollend entry next to slotchange and take the x-href off where I added it 👍🏻

@argyleink
Copy link
Author

cool, how's that @domenic?

Copy link
Member

@dlrobertson dlrobertson left a 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!

Copy link
Member

@domenic domenic left a 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.

@domenic domenic added topic: events addition/proposal New features or enhancements labels Oct 31, 2022
@dlrobertson
Copy link
Member

Implemented this in bug 1797013, so you can definitely add Gecko to implementers that are interested 😄

@argyleink
Copy link
Author

mdn issue added
noted firefox implementor interest
👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

addition/proposal New features or enhancements topic: events

Development

Successfully merging this pull request may close these issues.

[cssom-view] migration of the scrollend event did not include onscrollend

3 participants