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

Fix missing title on pdf and extension on cbz files #41

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LostQuasar
Copy link

No description provided.

@johnfactotum
Copy link
Owner

  • I don't think there's really a strong need to remove the file extension. Evince, for example, shows the whole file name in the title bar.
  • Not all files have extensions, and the part after . might be part of the title, though this is admittedly an edge case.
  • It's possible for files of other types to lack titles. So it might be better to handle this in the application code, or in <foliate-view>. The reason for dealing this in the application code is that in some cases one might need to know whether the book has a real title or not.

In sum, in my opinion it would be better to

  • Try to get real metadata from comic book files (e.g. ComicInfo.xml), and if not available, leave metadata empty.
  • In view.js, either always set missing title to filename, or expose the filename to the application.

@LostQuasar
Copy link
Author

LostQuasar commented Nov 4, 2024 via email

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