Adapt Animation events to new structure#12882
Conversation
Elchi3
left a comment
There was a problem hiding this comment.
Thanks, some change requests but looks good overall.
Aside: I see a lot of whitespace changes... I thought we updated many files recently to fix these things. Is something wrong with your editor settings or has mdn/content's editorconfig changed or something?
| oncancel = event => { } | ||
| ``` | ||
|
|
||
| ### Value |
There was a problem hiding this comment.
This "Value" section is not in the template. I don't think it is useful.
There was a problem hiding this comment.
Sorry forgot this one.
Co-authored-by: Florian Scholz <fs@florianscholz.com>
Co-authored-by: Florian Scholz <fs@florianscholz.com>
Co-authored-by: Florian Scholz <fs@florianscholz.com>
I modified them: they were unicode non-breakable spaces and I just replaced them with regular spaces. Some editors put them all over the place when you create new pages. (I think mainly Microsoft Word). |
This PR updates the
Animationinterface for new-style event pages. (see the template)The
on-*event handler pages are redirected to theeventname_eventpages.yarn content move Web/API/Animation/oncancel Web/API/Animation/cancel_eventyarn content move Web/API/Animation/onfinish Web/API/Animation/finish_eventyarn content move Web/API/Animation/onremove Web/API/Animation/remove_eventBCD: mdn/browser-compat-data#14914