-
Notifications
You must be signed in to change notification settings - Fork 15
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
[FR]: Support double-tap zoom #51
Comments
Thanks for the proposal. @herbeth1u However, is the 3-step behaviour like PhotoView necessary?
I checked the behavior of Google Photo, Twitter, Facebook, Discord, etc. |
How about making the number of steps configurable? If this is too overkill, settling on the 2-step behavior like most popular apps sounds like a reasonable plan as well. |
Noted. |
How is the state of this feature? The library is awesome, but I'm missing this functionality 😞 |
Thank you for waiting. I have been busy with my personal life recently and implementation has stopped. I will push the source code of the implementation in progress, so please take a look if you like. |
Is there an existing issue for this?
Describe the problem
As of now, Zoomable supports a "double-tap + drag" zoom. Is it possible to support the arguably more usual "double-tap" zoom?
For example, it's a feature supported by PhotoView or Discord's image viewer.
Describe the solution
How it works:
See https://github.com/Baseflow/PhotoView/blob/master/photoview/src/main/java/com/github/chrisbanes/photoview/PhotoViewAttacher.java#L231
Thanks.
The text was updated successfully, but these errors were encountered: