New issue
Advanced search Search tips

Issue 908758 link

Starred by 20 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Unable to prevent pinch-zoom gesture on Windows 10 laptop touchpad (MS Surface Pro laptop)

Reported by m.belya...@realtimeboard.com, Nov 27

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.84 Safari/537.36

Steps to reproduce the problem:
1. Open sample page with preventDefault():
http://jsfiddle.net/7Lk3e6am/5/show

2. Try to pinch to zoom with two fingers, so that one finger is still and another finger is moving away from it horizontally  

3. The viewport will be scaled after several attempts

What is the expected behavior?
The viewport does not scale

What went wrong?
Looks like some WheelEvents are not preventable even if preventDefault() was called on the event and Event.defaultPrevented is true

It looks like all such events has not zero deltaX property value.

Did this work before? N/A 

Chrome version: 70.0.3538.110  Channel: stable
OS Version: 10 Version 1803 (OS Build 17134.345)
Flash Version: -

Reproduced on MS Surface Pro laptop
 
Labels: Needs-Triage-M70
Components: -Blink Blink>Input
Cc: vamshi.kommuri@chromium.org
Labels: Triaged-ET Target-72 M-72 FoundIn-71 FoundIn-70 FoundIn-72
Status: Untriaged (was: Unconfirmed)
Thanks for filing the issue!

Able to reproduce the issue on reported chrome version 70.0.3538.110 and on latest canary 72.0.3624.0 using Windows 10 Surface Pro
Similar behaviour is seen from M60(60.0.3112.0) hence considering it as Non-Regression and marking it as Untriaged.
Cc: bokan@chromium.org
Owner: chaopeng@chromium.org
Status: Assigned (was: Untriaged)
Chao, did we ever implement sending a WheelEvent for preventing pinch-zoom with Windows trackpads?
Yes, we implemented wheel event for preventDefault for touchpad pinch zoom. I can not reproduce m.belyakov@'s behavior on Surface go on M70 and M72. Tested on  m.belyakov@'s test page and google map.

m.belyakov@ did you accidentally move mouse to the top bar which does not have preventDefault listener?

BTW, We will have async touchpad pinch zoom event on M71. After that developer must preventDefault the first event on pinch zoom sequence to cancel pinch zoom on touchpad.
Screenshot from 2018-11-29 15-21-08.png
3.9 KB View Download
Labels: Needs-Feedback
Hello Chao, Vamshi and Bokan!
Reproduced this bug after 20 or 30 attempts on this device:
https://www.microsoft.com/en-us/surface/devices/surface-book/overview

There is certain behaviour to reproduce this bug: you need to keep one finger still and move another finger as horizontally as possible. Also, you need to have patience)

>m.belyakov@ did you accidentally move mouse to the top bar which does not have preventDefault listener?
No, I didn't. Take a look at cursor position in this screencapture:
https://gyazo.com/05b5eb6316a3c6b6aba55932f56f2e16

P.S. I know about async pinch zoom event, so I prevent all mousewheel events in my demo.
Just found a easy way to reproduce this issue, quick scroll couple times then pinch.

I can reproduce on my wheel block test http://ht.chaopeng.me/wheel-block.html. So not related to OOPIF.

It may caused by MouseWheelEventQueue/TouchpadPinchEventQueue getting wrong reply. See the tracing.
trace_pinch.json.gz
2.1 MB Download
Also reproduced in MacOS (Mojave on 12-inch Macbook)

Sign in to add a comment