Skip to content

Add pinch-zoom token to touch-action property? #68

Closed

Description

@RByers

The touch-action property defines a number of tokens. Edge and Chrome also ship an additional pinch-zoom token (and I believe Firefox plans to as well /cc @staktrace). But it's not clear if this will ever make it into the W3C PointerEvents spec.

I think it's particularly important that we increase the visibility of this property (eg. document it on MDN for accessibility reasons. Many image carousels already use touch-action: pan-y when they probably should be using touch-action: pan-y pinch-zoom in order to avoid disabling zooming for vision impared users.

It'll be a little awkward to monkey patch it in, but is it OK if I submit a PR for this?
/cc @dtapuska @teddink @NavidZ @mustaqahmed

Activity

  1. RByers commented on Dec 20, 2016

    @RByers
    ContributorAuthor

    Concretely I believe the grammar should change from:
    auto | none | [ [ pan-x | pan-left | pan-right ] || [ pan-y | pan-up | pan-down ] ] | manipulation
    To:
    auto | none | [ [ pan-x | pan-left | pan-right ] || [ pan-y | pan-up | pan-down ] || pinch-zoom ] | manipulation

    I.e. pinch-zoom may optionally be specified along with the existing pan values (or by itself), but not in combination with auto, none or manipulation.

    manipulation then becomes a synonym for pan-x pan-y pinch-zoom.

  2. miketaylr commented on Dec 20, 2016

    @miketaylr
    Member

    Assuming this is something Gecko wants to ship eventually (or should otherwise be convinced to...), +1.

  3. graouts commented on Nov 13, 2018

    @graouts

    Is it expected that pinch-zoom will make it to a Level 3 spec for Pointer Events at W3C?

  4. RByers commented on Nov 13, 2018

    @RByers
    ContributorAuthor

    No such plans unfortunately since it's out of scope for the W3C PEWG charter. But I believe all PE implementations actually implement it, so it may be required for web compatibility (to avoid creating accessibility issues).

  5. NavidZ commented on Nov 13, 2018

    @NavidZ

    There is a test for it that we can add it to wpt as well under something like "de-facto" behaviors but not part of any specification.

  6. RByers commented on Nov 13, 2018

    @RByers
    ContributorAuthor

    Oh yeah, it would be great to upstream our test! It doesn't need to be "defacto" - this IS specified in the compat spec, so you can add it https://github.com/web-platform-tests/wpt/tree/master/compat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions