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

Should DragEvent be upgraded to a PointerEvent? #106

Closed
RByers opened this issue Jul 13, 2016 · 4 comments
Closed

Should DragEvent be upgraded to a PointerEvent? #106

RByers opened this issue Jul 13, 2016 · 4 comments

Comments

@RByers
Copy link
Contributor

RByers commented Jul 13, 2016

Split off from #100. Unlike click etc., Edge still uses MouseEvent for dragstart etc.

@smaug----
Copy link
Contributor

smaug---- commented Jul 13, 2016

MouseEvent for dragstart? Are you sure about that? Wouldn't dnd be rather broken if DragEvent wasn't used.
Isn't upgrading existing events actually a bit regression risky. That changes the prototype chain.
Though, I guess we want to support dnd somehow using touch.

@RByers
Copy link
Contributor Author

RByers commented Jul 13, 2016

Sorry, what I meant to say is that DragEvent inherits just from MouseEvent, not PointerEvent.

Yes, there's definitely some compat risk here. Is it much worse than in #100 though (which apparently Edge has already shipped without issue)?

@NavidZ
Copy link
Member

NavidZ commented Mar 28, 2018

We talked about this in the call. There are some advantages to this such as having pointerId and pointerType for multiple dragging at the same time but there are also some complications like handling of setPointerCapture. Pushing this to V3 to further discuss this.

@patrickhlauke
Copy link
Member

patrickhlauke commented Aug 19, 2020

Discussed in the PEWG call today https://www.w3.org/2020/08/19-pointerevents-minutes.html. @smaug---- filed issue against HTML spec. Closing issue here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants