Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a note about the 300ms click delay #20

Merged
merged 1 commit into from
Sep 21, 2015

Conversation

RByers
Copy link
Contributor

@RByers RByers commented Sep 11, 2015

Talking about specific gestures is out of scope for the pointer events charter / specification. But hopefully we can at least have a note that describes the important impact touch-action has on the click delay? Here I attempt to do that without even using the word "zoom", WDYT?

See related twitter discussion: https://twitter.com/awfulben/status/642163198511026176

@RByers
Copy link
Contributor Author

RByers commented Sep 11, 2015

@jacobrossi WDYT?

@jacobrossi
Copy link
Member

Good idea. Can you s/tap gesture/click event?

@RByers
Copy link
Contributor Author

RByers commented Sep 11, 2015

Thanks, how's this? (I'll squash all commits together before merging - don't need the intermediate history here).

@patrickhlauke
Copy link
Member

From a reader's point of view, this version is a bit unsatisfactory as it doesn't make it clear why user agents must add the 300ms delay. perhaps after <code>click</code> event we could have to account for the possibility of a subsequent tap which may trigger further default behavior? still a very roundabout way of saying what could really be said in four simple words, though...

patrickhlauke referenced this pull request in RByers/w3c-pointerevents Sep 11, 2015
@RByers
Copy link
Contributor Author

RByers commented Sep 11, 2015

Seems like something like that could work. It could even be "subsequent touch action" - eg. Chrome has double-tap-drag (tap and then drag to zoom) which can have the same issue as double tap (though strangely doesn't today).

@patrickhlauke
Copy link
Member

perhaps also change often must to typically

@jacobrossi
Copy link
Member

Let me run this by some folks on my side to see how flexible we can be. :-)

@RByers
Copy link
Contributor Author

RByers commented Sep 11, 2015

perhaps also change often must to typically

done

I also added a bit more that should be uncontroversial: "(to resolve ambiguity around possible subsequent touches)".

Let me run this by some folks on my side to see how flexible we can be. :-)

Ok. Let me know if that's likely to take more than a couple days and we can land the uncontroversial wording for now, then hope to elaborate later :-)

@jacobrossi
Copy link
Member

We can use Patrick's proposed suggestions (including calling double-tap / tap out specifically for clarity) since that would make this clearer. Perhaps we can add something like this just to make it clear what's out of scope?

Methods for determining a tap or double-tap gesture are out of scope for this specification.

@patrickhlauke
Copy link
Member

:shipit:

@RByers
Copy link
Contributor Author

RByers commented Sep 17, 2015

Great, so like this then? (I squashed all the commits)

@RByers
Copy link
Contributor Author

RByers commented Sep 21, 2015

@patrickhlauke @jacobrossi let me know if either of you want to review the final text here before I merge it. Otherwise it sounds like it's safe to merge and tweak later as desired.

@patrickhlauke
Copy link
Member

If I can be a right wordy pain in the neck... other behavior > other behaviors. Also, after mentioning the 300ms delay and auto, the note seems to just hang there and not follow through with the next part...that you can disable that delay. And although we're already in a note, that last sentence may need a bit of a lead-in. How's this:

<section class="note">Disabling some default touch behaviors may allow user agents to respond to other behaviors more quickly.  For example, with <code>auto</code> user agents typically add 300ms of delay before <code>click</code> to allow for double-tap gestures to be handled.  In these cases, explicitly setting <code>touch-action: none</code> or <code>touch-action: manipulation</code> will remove this delay. Note that the methods for determining a tap or double-tap gesture are out of scope for this specification.</section>

@RByers
Copy link
Contributor Author

RByers commented Sep 21, 2015

Thanks, I like that better (I was definitely getting sloppy) - using it verbatim. I think there's been enough discussion now that this can land, but happy to take more feedback in followup commits.

RByers added a commit that referenced this pull request Sep 21, 2015
Add a note about the 300ms click delay
@RByers RByers merged commit 6c17eee into w3c:gh-pages Sep 21, 2015
@RByers RByers deleted the click-delay-note branch September 21, 2015 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants