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

Change click/auxclick/contextment/dblclick event types to PointerEvent? #235

Closed
NavidZ opened this issue Jun 24, 2019 · 1 comment · Fixed by #259
Closed

Change click/auxclick/contextment/dblclick event types to PointerEvent? #235

NavidZ opened this issue Jun 24, 2019 · 1 comment · Fixed by #259

Comments

@NavidZ
Copy link
Member

NavidZ commented Jun 24, 2019

@garykac @travisleithead @BoCupp-Microsoft @patrickhlauke

We've seen some need to additional information for the meta events that are generated for different pointers. Attributes such as pointerType seems to be useful for some web apps as mentioned in this issue file against PointerEvent repository.
So we have two paths:

  1. Either add all the attributes required back to the MouseEvent idl.
  2. Just change these meta events (i.e. click, auxclick, contextmenu, and dblclick) type to the PointerEvent.

I wonder how ui events folks feel about the second solution. From PointerEvents perspective that seems to be a better long term solution but obviously we need to evaluate the compat impact of that.

@patrickhlauke
Copy link
Member

noting that one of the compat issues may be the fact that in Pointer Events, the coordinates are fractional - likely the biggest potential problem, as scripts likely won't expect that

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

Successfully merging a pull request may close this issue.

2 participants