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

EPUB: Previous/Next buttons should follow content.opf, not toc.ncx #37

Open
brjsp opened this issue Dec 8, 2024 · 3 comments
Open

EPUB: Previous/Next buttons should follow content.opf, not toc.ncx #37

brjsp opened this issue Dec 8, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@brjsp
Copy link

brjsp commented Dec 8, 2024

Some EPUB files have chapters which are not referenced by any anchor in toc.ncx. This causes missing/inaccessible content in kchmviewer.

@u-235
Copy link
Member

u-235 commented Dec 9, 2024

Hi. Can you download such an ebook for the tests?

@brjsp
Copy link
Author

brjsp commented Dec 9, 2024

https://libgen.li/ads.php?md5=6395F862507F37848EFAEDBB27968F7A

@u-235
Copy link
Member

u-235 commented Dec 9, 2024

Thank you, it's clear what's going wrong.

Historically kchmviewer uses the table of contents for jumps using the "Next" and "Previous" buttons, most likely this is due to the lack of an official CHM specification. And there is no special code yet for navigating using the navigation map.

Conclusions:

  • Add code for navigation map processing.
  • Change the action of the "Home" button to show the title page instead of the first page in the table of contents.
  • It would be useful to add an indication of the presence of hidden pages.

The goal is clear, but I don't have time to implement it yet.

@u-235 u-235 added the bug Something isn't working label Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

2 participants