Demix transition events from GlobalEventHandlers #16758
Conversation
github-actions
bot
added
the
data:api 🐇
queengooborg
added
the
needs content update 📝
|
This pull request has merge conflicts that must be resolved before it can be merged. |
|
This pull request has merge conflicts that must be resolved before it can be merged. |
|
This pull request has merge conflicts that must be resolved before it can be merged. |
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
|
This pull request has merge conflicts that must be resolved before it can be merged. |
|
This pull request has merge conflicts that must be resolved before it can be merged. |
api/Element.json
Outdated
| { | ||
| "version_added": "12", | ||
| "partial_implementation": true, | ||
| "notes": "The event handler exists but will never be called." |
There was a problem hiding this comment.
Can you match the wording we have elsewhere? "Although the onmessageerror property is supported, the messageerror event is never fired. See bug 171216."
|
This pull request has merge conflicts that must be resolved before it can be merged. |
| "__compat": { | ||
| "description": "<code>transitionend</code> event", | ||
| "mdn_url": "https://developer.mozilla.org/docs/Web/API/Element/transitionend_event", | ||
| "spec_url": "https://drafts.csswg.org/css-transitions/#transitionend", |
There was a problem hiding this comment.
Just one spec URL here? Can you do the same as above?
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
queengooborg
removed
the
needs content update 📝
|
@queengooborg there were comments not resolved yet in #16758 (comment) |
This PR demixes the
transitionevents from the GlobalEventHandlers mixin. There was a lot of inconsistency in the original data, a resolution was attempted here.