-
-
Notifications
You must be signed in to change notification settings - Fork 191
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
Support pinch-zoom #645
Comments
Thank you for your feature request @kj. Embla doesn’t prevent this but the examples all have This might only be a matter of finding the .classname {
touch-action: pan-y pinch-zoom;
} Let me know how it goes. Best, |
Hi David, thanks for your quick response. I completely forgot to mention that I was testing on Firefox for Android. I just tried Chrome (actually Cromite) and it works there, even on the example page, which is great! I did actually try earlier with |
@kj I don’t know anything about Keen-Slider but you can add a callback to the Embla watchDrag option to customize the behavior and intercept the drag behavior. In your case, you need to do the following:
Best, |
Oh that does sound like it should work. I'll try it out tomorrow probably and report back. |
@kj do you intend to pursue this? |
Sorry @davidjerleke I didn't forget about this, I've just been a bit distracted. I tried with a minimal setup, basically this:
And pinch zoom does work in Firefox for Android with It's great to finally have a lightweight carousel that just does these things right! |
Feature request is related to
Is your feature request related to an issue?
Pinch zoom isn't working for me on Android (at least in the examples on the website). I noticed in another issue #480 (comment) someone commented that it is working under iOS, so this might only affect Android, although I can't be sure.
Describe the solution you'd like
I'd like to be able to use pinch zoom to zoom in to image details in a slider. I made a PR for Keen-Slider to get this working, although there's not much development activity over there so the PR hasn't gone anywhere.
The PR just checks whether there are two or more fingers touching the surface before dragging, and adds
pinch-zoom
totouch-action
.The text was updated successfully, but these errors were encountered: