Closed
Bug 1710317
Opened 3 years ago
Closed 3 years ago
Pen eraser input emits pointer events with .button=0 on Windows
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P3)
Tracking
()
RESOLVED
FIXED
91 Branch
Tracking | Status | |
---|---|---|
firefox91 | --- | fixed |
People
(Reporter: saschanaz, Assigned: saschanaz)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
https://searchfox.org/mozilla-central/source/widget/windows/nsWindow.cpp#8675
There is no check for erasers. This is not a blocker for bug 1707080 since it's already broken without it.
Assignee | ||
Comment 1•3 years ago
•
|
||
MOZ_SOURCE_ERASER should be removed since it has no use and Pointer Events spec considers erasers as pens.
Updated•3 years ago
|
Severity: -- → S3
Priority: -- → P3
Assignee | ||
Comment 2•3 years ago
|
||
Updated•3 years ago
|
Assignee: nobody → krosylight
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•3 years ago
|
||
It's not ideal to have mButton[s] only to pass them to PointerEvent, but it's required anyway.
Depends on D115508
Updated•3 years ago
|
Attachment #9222657 -
Attachment description: Bug 1710317 - Part 2: Add mButton[s] to WidgetTouchEvent r=aklotz → Bug 1710317 - Part 2: Add mButton[s] to WidgetTouchEvent r=aklotz,edgar
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9d16301a3675 Part 1: Add MouseButton::eEraser r=aklotz,edgar https://hg.mozilla.org/integration/autoland/rev/d92899c7f5cc Part 2: Add mButton[s] to WidgetTouchEvent r=aklotz,edgar
Comment 5•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9d16301a3675
https://hg.mozilla.org/mozilla-central/rev/d92899c7f5cc
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox91:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•