New issue
Advanced search Search tips
Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Legal: ----
Launch-M-Approved: 56-Stable
Launch-M-Target: 56-Stable
Launch-Privacy: ----
Launch-Security: ----
Launch-Status: ----
Launch-Test: ----
Launch-UI: ----
Product-Review: ----

Blocking:
issue 639227



Sign in to add a comment

Add 'pinch-zoom' support to touch-action

Project Member Reported by rbyers@chromium.org, Jul 6 2016 Back to list

Issue description

Add support for the 'pinch-zoom' token to the touch-action CSS property.

proposed spec here: https://rawgit.com/RByers/w3c-pointerevents/touch-action-pinch-zoom/index.html#the-touch-action-css-property

Discussion: https://github.com/w3c/pointerevents/issues/29

I think it's possible that landing this in the spec may get bogged down by politics, but I don't think that should stop us.  IE/Edge have long-shipped this: https://msdn.microsoft.com/en-ca/library/windows/apps/hh767313.aspx.  So we can just ship compatiblity as a "de facto standard".

I think this is urgent because there are scenarios where we'd want people to use passive event listeners (eg. image carousels) where we really don't want to force them into hurting accessibility of their site by disabling pinch-zoom.

See, for example, https://bugs.jqueryui.com/ticket/14993

I think this should be trivial to implement - IIRC all the plumbing is already there due to 'touch-action: manipulation'.  We just have to hook up CSS parser syntax to the WebTouchActionPinchZoom bit.

Ideally we'd also land a web-platform-test for this, but I wouldn't block shipping on it.

dtapuska@ WDYT, worth trying to do in M54?


 
I think it is relatively trivial to do. The main bulk of the work is writing a test. Definitely something doable for M54.

We should do this work alongside with  issue 626101 
Owner: dtapu...@chromium.org
Status: Assigned

Comment 3 by rbyers@chromium.org, Aug 23 2016

Blocking: 639227
Note that there's some benefit to getting this out before the intervention in  issue 639227 .  That intervention will force some developers to use touch-action on their image carousels, and it would be a better user experience (and better for accessibility) if we could recommend 'touch-action: pan-y pinch-zoom' instead of just 'touch-action: pan-y' for those scenarios.

It's not a hard dependency though.
Components: Blink>Input
Labels: -M-54 M-55
AMP needs this feature: https://github.com/ampproject/amphtml/issues/4820#issuecomment-245992277
Labels: -M-55 M-56 Launch-M-Approved-56-Stable Launch-M-Target-56-Stable
Status: Fixed

Comment 9 Deleted

Sign in to add a comment