-
Notifications
You must be signed in to change notification settings - Fork 72
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
Add catalog example for changing pages #98
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
React part looks good 👍
index.js
Outdated
@@ -188,7 +188,6 @@ PSPDFKitView.propTypes = { | |||
* fragmentTag: A tag used to identify a single PdfFragment in the view hierarchy. | |||
* This needs to be unique in the view hierarchy. | |||
* | |||
* @platform android |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you accidentally removed this from the wrong doc. Should be removed from onStateChanged
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed! Nice Catch! Fixed.
6c3aebd
to
7c5db10
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍
samples/Catalog/Catalog.ios.js
Outdated
configuration: { | ||
useParentNavigationBar: true, | ||
}, | ||
style: { flex: 1 }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you don't need to pass any props here, since we are not directly using a PSPDFKitView
.
The iOS implementation for #87