Skip to content
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

Zoom doesn't work with double-tapes #781

Open
houdaichang opened this issue Feb 1, 2021 · 0 comments
Open

Zoom doesn't work with double-tapes #781

houdaichang opened this issue Feb 1, 2021 · 0 comments

Comments

@houdaichang
Copy link

🐛 Bug Report

Expected behavior

Reproduction steps

Configuration

Version: 1.x

Platform: 🤖 Android 9.x
Sometimes with a double-tap on the view, the zoom does not work.
After downloading the library and spreading the logs, I found a place where the code behaves differently than with a successful response.
PhotoViewAttacher -> OnGestureListener onGestureListener -> onScale
if (getScale() < mMaxScale || scaleFactor < 1f) {...}
This condition does not always work with double-tapes. Zoom does not work.
However, if you double-tap again, the zoom will work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant