Issue metadata
Sign in to add a comment
|
stopPropagation() doesn't work in 55.0.2883.84
Reported by
pavel.va...@motorolasolutions.com,
Dec 7 2016
|
||||||||||||||||||||||
Issue descriptionDevice name: Nexus 6P Application version: 55.0.2883.84 Operating system: Android 6.0.1 Steps to reproduce: (1) Create 2 divs: one inside the other. (1) Assign event listener using addEventListener(event, function, true) on "touchstart", "touchmove" or "touchend" events in both divs. Add log message in all event listeners. (2) Add stopPropagation() in the event listener of parent div. (3) Check if event goes to child div. Expected result: stopPropagation() works. Event doesn't go to the other div. Actual result: stopPropagation() doesn't work. It worked on WebView 54.0.2840.85
,
Dec 7 2016
,
Dec 7 2016
Looking into it now.
,
Dec 7 2016
,
Dec 7 2016
On my Nexus 7 (Android 6.0.1), both Chrome beta (55.0.2883.77) and Chrome dev (56.0.2924.18) work fine. Repro: http://output.jsbin.com/revabo pavel.vasiliev@motorolasolutions.com: Could you please check if my repro captures your case, and if it works on your 6P?
,
Dec 8 2016
Hmm, your code works, but this one doesn't work on the latest WebView and works with previous: Code: http://jsbin.com/xoyeriqovi/edit?html Page: http://output.jsbin.com/xoyeriqovi I tried even with Chrome DevTool toggling it to device mode (Toggle device toolbar button). And it passes events in my code.
,
Dec 8 2016
In your repro, the map still pans because the arcgis/esri code uses the new PointerEvent API (instead of touch events) whenever available. Chrome enabled PointerEvents by default on M55, so stopped touch* events has no effect. Stopping the propagation of "pointerdown", "pointermove" and "pointerup" events works. I think you will want this in addition to your current "touch*" events for best compat.
,
Dec 8 2016
Ohh, thank you very much!
,
Jan 5 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dtapu...@chromium.org
, Dec 7 2016Owner: mustaq@chromium.org
Status: Assigned (was: Unconfirmed)