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 bookmark support #226

Merged
merged 12 commits into from
Dec 23, 2023
Merged

✨ Epub bookmark support #226

merged 12 commits into from
Dec 23, 2023

Conversation

aaronleopold
Copy link
Collaborator

Relates to #115

PR adds basic support for bookmarks. The DB is ready to support bookmarks outside of epub files, however the UI has not been developed for other media types/readers.

Also reorganized a little bit, breaking up really large files (e.g. db::entity::media) into separate, more focused files.

Fixed a bug with the chapterName determination, refreshing the top and bottom bars for epub controls
If an entity file was excessively large, I just decided to split it up for easier navigation through the code
I still do not feel it is there yet, but it is a bit better
Toggling works, but the behavior is very naive and dependent on viewport uniformity
@aaronleopold aaronleopold changed the title [WIP] ✨ Epub bookmark support ✨ Epub bookmark support Dec 23, 2023
@aaronleopold aaronleopold marked this pull request as ready for review December 23, 2023 17:03
@aaronleopold aaronleopold merged commit 5789102 into develop Dec 23, 2023
7 checks passed
@aaronleopold aaronleopold deleted the al/epub-bookmarks branch December 23, 2023 17:05
@aaronleopold aaronleopold mentioned this pull request Feb 18, 2024
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.

1 participant