-
Notifications
You must be signed in to change notification settings - Fork 38
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 swipe/slide/zoom functionality #36
Comments
@rejas Just for the record, there is no interest in adding dependencies, correct? |
No, only if it was very very small. For exmaple issue #47 would be really easy to fix if I would use hammer.js but then the size of the library doubles.... |
@rejas just to update, mobile finger sliding works just fine. It's only trackpad that does not. |
Trackpads? Well, for those that want to add support for that, they could use their own library for detecting that an call a function for next / prev on our library. what do you think? |
For me zooming doesn't work on Pixel XL with Android 9 and Chrome 86.0.4240.198 Do you have to configure it the right way to work? Or did you mean sliding next/prev and not zooming? |
From "paxperscientiam": On the UX side of things, the ability to slide over to the next image via trackpad or mobile screen, would be great. To this end, I think we want a sliding motion to be equivalent to pressing left/right buttons, correct?"
Correct :-)
The text was updated successfully, but these errors were encountered: