-
Notifications
You must be signed in to change notification settings - Fork 297
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
'Retargeting steps' run at wrong time #561
Comments
This also affects w3c/uievents#157. |
Here's an idea:
I think this works since relatedTarget and touch events are mutually exclusive. Even if they weren't though, this could still work by relatedTarget just occupying a special slot. The complexity here is that UI Events and Touch Events need to initialize relatedTargets properly, but that's all the complexity they need to worry about. Everything else can be done by the DOM Standard. (I was initially thinking of handling cc @whatwg/components |
Ah, this does require event objects that support related targets to implement "initialize related target steps" which is passed path's relatedTargets. That way we can update what gets exposed. That still seems reasonable. |
Actually, even that isn't necessary if they just make those attributes reflect the internal relatedTargets slot. That is a little special for events and might require some constructor special casing though. |
This makes it work for touch events. Tests: ... Fixes part of #561.
This makes it work for touch events. Tests: ... Fixes part of #561.
This makes it work for touch events. Tests: ... Fixes part of #561.
All the retargeting should happen before event is being handled, since event handling may change the underlying DOM and that way affect to the retargeting.
Touch event specific bug, w3c/touch-events#92
The text was updated successfully, but these errors were encountered: