[Pointer Events] Duplicated pointerdown event in multi touches
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P2)
Tracking
()
People
(Reporter: edgar, Assigned: edgar)
References
Details
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
text/x-phabricator-request
|
lizzard
:
approval-mozilla-beta+
|
Details |
Assignee | ||
Comment 1•6 years ago
|
||
STR:
- Press screen with the first finger (do not release it)
- Press with another finger.
Actual results:
A duplicated pointerdown event is fired for first press.
Expected results:
There should only one pointerdown event fired for first press.
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Comment 3•6 years ago
|
||
Comment 4•6 years ago
|
||
I wonder why (manual) wpt tests for pointer events didn't catch this.
Updated•6 years ago
|
Comment 6•6 years ago
|
||
bugherder |
Comment 8•6 years ago
|
||
This is a major improvement for ARM arch and Windows, I think it's worth uplifting to 66.
Edgar, can you please prepare a uplift request?
Assignee | ||
Comment 9•6 years ago
|
||
Comment on attachment 9039686 [details]
Bug 1521082 - Don't dispatch pointer event if the touch in touchstart event is already registered;
Beta/Release Uplift Approval Request
Feature/Bug causing the regression
None
User impact if declined
The pinch to zoom in/out on google map on touch screen device doesn't work, it always goes to maximum zoom.
It is a really bad experience on google map on touch screen devices.
Is this code covered by automated tests?
Yes
Has the fix been verified in Nightly?
Yes
Needs manual test from QE?
No
If yes, steps to reproduce
List of other uplifts needed
None
Risk to taking this patch
Low
Why is the change risky/not risky? (and alternatives if risky)
The changes is small, only affects pointer event on touch screen device and also make our behavior align with other browsers.
String changes made/needed
None
Comment on attachment 9039686 [details]
Bug 1521082 - Don't dispatch pointer event if the touch in touchstart event is already registered;
Fix for pinch to zoom on touch screens, let's uplift for beta 6.
Is this something that never worked correctly in this case, or, is it a regression?
Or, maybe this site changed recently?
Thanks for adding new tests, too!
Comment 12•6 years ago
|
||
bugherder uplift |
Assignee | ||
Comment 13•6 years ago
|
||
(In reply to Liz Henry (:lizzard) (use needinfo) from comment #10)
Is this something that never worked correctly in this case, or, is it a regression?
Or, maybe this site changed recently?
I am not sure if site changed recently.
For pointer event, this is not a regression, it is something that never worked correctly.
Updated•6 years ago
|
Description
•