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

Added custom readingOrder to EPubNavigationViewController #332

Conversation

chrfalch
Copy link
Contributor

To be able to show non-linear resources this commit adds an optional and extra parameter to the EPubNavigationViewController which is by default set to be the publication's readingOrder.

This lets us show non-linear resources or navigate to internal resources in separate views. This can be used to solve issues like Impossible to open resource with linear=no #330.

… EPubNavigationViewController

To be able to show non-linear resources this commit adds an optional and extra parameter to the EPubNavigationViewController which is by default set to be the publication's readingOrder.

This lets us show non-linear resources or navigate to internal resources in separate views. This can be used to solve issues like Impossible to open resource with linear=no readium#330.
CR: Removed adding readingOrder to deprecated constructor

Co-authored-by: Mickaël Menu <mickael.menu@gmail.com>
…it instead of setting it to publication.readingOrder.
@mickael-menu mickael-menu temporarily deployed to LCP August 29, 2023 10:20 — with GitHub Actions Inactive
@mickael-menu mickael-menu temporarily deployed to LCP August 29, 2023 10:20 — with GitHub Actions Inactive
Copy link
Contributor Author

@chrfalch chrfalch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, a lot more clear than my implementation :)

Copy link
Member

@mickael-menu mickael-menu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for tackling that @chrfalch!

@mickael-menu mickael-menu merged commit e3056c6 into readium:develop Aug 29, 2023
5 of 6 checks passed
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

Successfully merging this pull request may close these issues.

2 participants