-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Spec the prefixed CSS Animation/Transition event handlers #5188
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
Spec the prefixed CSS Animation/Transition event handlers #5188
Conversation
@domenic @annevk PTAL. Reminder: we had previous conversations in whatwg/compat#119 :) |
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.
Normative content seems good, just some editorial stuff...
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.
Looks great. Will leave for a bit in case @annevk wants to take a look; otherwise will merge tomorrow.
Note that via the IDL sync this will automatically generate tests for GlobalEventHandlers in https://github.com/web-platform-tests/wpt/blob/master/html/webappapis/scripting/events/event-handler-all-global-events.html. @stephenmcgruer can you comment back on this issue with the bug URLs for each browser? |
@domenic Sorry, what bug URLs are you referring to? |
@stephenmcgruer bugs you filed against browsers, such as https://bugzilla.mozilla.org/show_bug.cgi?id=1607918 against Firefox. We typically list those in specification PRs for people doing spelunking. |
Ack. Here are the related bugs from this spec PR change: Chrome:
Firefox:
Safari:
|
See whatwg/compat#118
Tests: web-platform-tests/wpt#19099 (will need moved from compat to appropriate parts of the html spec).
/webappapis.html ( diff )