fix: change event types from MouseEvent to PointerEvent #2016
Conversation
|
Thanks for the PR! This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged. |
|
What do you think @saschanaz |
1 similar comment
|
What do you think @saschanaz |
| @@ -196,10 +196,6 @@ | |||
| "name": "dblclick", | |||
| "type": "MouseEvent" | |||
| }, | |||
| { | |||
| "name": "contextmenu", | |||
| "type": "MouseEvent" | |||
There was a problem hiding this comment.
This and auxclick should also be PointerEvent. https://w3c.github.io/pointerevents/#the-click-auxclick-and-contextmenu-events
There was a problem hiding this comment.
Hmmmmm actually contextmenu and auxclick already has PointerEvent? 🤔 In that case we can remove contextmenu and auxclick here.
There was a problem hiding this comment.
Actually, no, they were being overwritten in the overridingTypes, I have just removed them from there.
|
LGTM |
|
There was an issue merging, maybe try again saschanaz. Details |
|
LGTM |
|
Merging because @saschanaz is a code-owner of all the changes - thanks! |
I have converted MouseEvent to PointerEvent in some occurrences.
References:
1- https://w3c.github.io/uievents/#event-type-click
2 - https://caniuse.com/?search=PointerEvent
3 - https://developer.mozilla.org/en-US/docs/Web/API/Element/click_event
Solves:
1- microsoft/TypeScript#60746
2 - microsoft/TypeScript#61647 (comment)