-
Notifications
You must be signed in to change notification settings - Fork 189
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
enablePinchOnMobile set to false completely disables pinch to zoom on mobile #914
Comments
Which operating system are you using? I just tried with OSX, and it seems to be OK. |
This is on iOS. I tried directly on chrome and a react-native webview which I guess is iOS webkit. |
THX! |
Oops. I've managed to reproduce the bug on my iPad. I didn't expect that! |
Here's the culprit: mozilla/pdf.js#12212 BTW, zooming isn't entirely disabled. You just have to pull down the page with on finger before starting to pinch. |
The bugfix has landed with version 10.0.0. |
The description of enablePinchOnMobile says that the default behavior is to zoom the entire window but this isnt happening with enablePinchOnMobile set to false. For some reason, it seems that zooming is somehow completely disabled.
The reason Im playing with this variable is enablePinchOnMobile zooming seems to be completely broken (PDF.js 2.9 + v9.0 of library) but thats a whole other issue. Im looking to get basic zooming working as quick as I can.
The text was updated successfully, but these errors were encountered: