Skip to content
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

Second chance to find respectively type of event handler #1184

Closed
dSalieri opened this issue Apr 15, 2023 · 5 comments
Closed

Second chance to find respectively type of event handler #1184

dSalieri opened this issue Apr 15, 2023 · 5 comments

Comments

@dSalieri
Copy link

dSalieri commented Apr 15, 2023

  1. If event’s type attribute value is a match for any of the strings in the first column in the following table, set event’s type attribute value to the string in the second column on the same row as the matching string, and return otherwise.
Event type Legacy event type
"animationend" "webkitAnimationEnd"
"animationiteration" "webkitAnimationIteration"
"animationstart" "webkitAnimationStart"
"transitionend" "webkitTransitionEnd"

I saw this table not once, but I really can't understand why is there only 4 names of event handlers?

Why don't we see for example "transitionstart" or "transitioncancel" and so on?

Need clarification about this movement in direction these names.

@annevk
Copy link
Member

annevk commented Apr 18, 2023

Did you look in blame for context? Did you write a test for the other names you expect?

@dSalieri
Copy link
Author

dSalieri commented Apr 18, 2023

@annevk Idk where I can look blame of this table. I checked out webkitTransitionStart, webkitTransitionCancel and I didn't find anything. So I can conclude we have only 4 props because we had no others valid in past time.

@annevk
Copy link
Member

annevk commented Apr 19, 2023

https://github.com/whatwg/dom/blame/main/dom.bs (or in your local checkout).

@dSalieri
Copy link
Author

dSalieri commented Apr 19, 2023

@annevk oh thanks, I really tried to find how to watch blame. I've read everything what related with it. And my opinion haven't changed: in the past existed these 4 events with webkit prefix and now events with webkit additionaly supported for compatibility.

@bathos
Copy link

bathos commented Apr 19, 2023

@dSalieri I’m not 100% sure I get what you’re asking, but there are only four because those were the only four needed to address the motivating compat issue. Ideally, there would be none.

@annevk annevk closed this as not planned Won't fix, can't repro, duplicate, stale Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants