-
Notifications
You must be signed in to change notification settings - Fork 51
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
[Desktop] swipe event fires on incorrectly desktop and non-touch devices #99
Comments
Yes you are right, and this is the desired result. |
Can you explain why that makes sense, so we can have a better understanding of the intention? The expectation for our users is that on mobile, a swipe touch event is necessary, but on desktop this breaks the whole experience. When a user on desktop attempts to copy text from the screen, the swipe activates, which is a huge problem. |
I got your problem. How about add a new option to disable swipe on desktop? Or anything better? |
Works for me, I'd honestly make it the default. There's something confusing about drag versus swipe. But I haven't really dug into all the details. Would this have to be a new v- type or a some property added to the end of swipe... |
I prefer to make things common, so I think |
My advice would be for this not to be an option at all. In reality the |
Swipe events should never fire on devices that aren't touch screens.
The text was updated successfully, but these errors were encountered: