-
Notifications
You must be signed in to change notification settings - Fork 555
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
No member named 'enableDoubleTapZoom' in 'facebook::react::RNPDFPdfViewProps' #878
Comments
I have the same issue. |
I'm facing the same issue RN "0.76.1" |
same issue |
2 similar comments
same issue |
same issue |
Please react to the original issue with 👍 instead of commenting. |
As a temporary workaround, or with a patch-package fix, you may change the following line in:
Then re-run // ...
bool enableAntialiasing{false};
bool enableDoubleTapZoom{false}; // this will now have been added if the codegen works
int fitPolicy{0};
// ... |
@scarlac sorry for being inpatient, but do you by chance know when can we expect the new version of package to be released with this fix? |
@scarlac Double-clicking on the PDF viewer in iOS causes the app to crash. It seems that additional fixes may be needed to apply a proper patch to the package. |
Sorry, I'm not the maintainer of this repo, so I don't know. I had other issues while migrating so I eventually gave up, so it's possible my patch isn't sufficient, as some are indicating. |
Commenting out line 735 in This gives you the added "feature" of selecting text by a tap followed immediately by a drag. (Maybe it was already like that?) |
Created pull request for fix |
What
react-native
version are you using? 0.76.1What
react-native-pdf
version are you using? 6.75What platform does your issue occur on? ios
Describe your issue as precisely as possible :
In migration process to RN 0.76.1 this brokes, works ok in RN 0.73.9
The text was updated successfully, but these errors were encountered: