Skip to content

Add noMultiTouch option to prevent reacting to multi touch events - #177

Open
cburgdorf wants to merge 2 commits into
jakiestfu:masterfrom
sofa:master
Open

cburgdorf wants to merge 2 commits into
jakiestfu:masterfrom
sofa:master

Conversation

@cburgdorf

Copy link
Copy Markdown

This adds a new setting noMultiTouch and makes it the default.
With noMultiTouch set to true snap.js won't react to multi
finger gestures. This should be the default as it's quite unlikely
to be what the user expects. It helps to prevent issues
with other gestures like pinch zooming.

This adds a new setting noMultiTouch and makes it the default.
With `noMultiTouch` set to true snap.js won't react to multi
finger gestures. This should be the default as it's quite unlikely
to be what the user expects. It helps to prevent issues
with other gestures like pinch zooming.
@jakiestfu jakiestfu added this to the v2.0 milestone Mar 9, 2014
@jayd3e

jayd3e commented Mar 10, 2014

Copy link
Copy Markdown
Collaborator

I would say that Snap.js shouldn't make this an option, it just need to be the native behavior. The gesture we are going for is a single finger swipe to the left or right, no? So if that's the gesture that is supported, we need to stick with that. If we allow a multi-finger swipe, then we need to account for that. I just don't see this as something that should get turned on/off.

@cburgdorf

Copy link
Copy Markdown
Author

That's why I made it the default because I agree that a single finger swipe is what we aim for. I made it an option because I didn't know if people may rely on multi-touch swipes as that's the current behavior.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants