Skip to content

BookMap: visual tweaks and enhancements #10530

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

Merged
merged 5 commits into from
Jun 4, 2023

Conversation

poire-z
Copy link
Contributor

@poire-z poire-z commented Jun 2, 2023

Discussed all along #10433. Closes #10433.

PageBrowser: smaller margins if no thumbnail page numbers

Also fix possibly unbalanced left and right margins.

BookMap: nicer alignment of the vertical page number

Align page numbers (the vertical ones, on the left of a BookMapRow) on the baseline of the BookMapRow, where page slots rise from.
Also tweak the page number spike below the baseline used in PageBrowser.

BookMap: add option "Page browser on tap"

Enabled by default, to get the current behaviour.
Disabling it allows people with small fingers to jump directly to that page in Reader.

BookMap: add option "10-page markers"

Allows hairy-boxes amateurs to get small markers (under the baseline) every 10 pages (with -/+ to get small or medium markers, with optionally smaller ones every 5 pages).

BookMap: add "Overview" mode

Add a restricted but convenient mode showing BookMap like in initial view, while still allowing chapter levels to be tweaked. This allows getting back to this view with another gesture to see the overall progress in the book, while still having the normal BookMap in flat mode acting as an alternative ToC.
Available as an action to associate to a gesture, and with long-press on the "Book map" menu item.


This change is Reviewable

poire-z added 5 commits June 2, 2023 20:44
Also fix possibly unbalanced left and right margins.
Align page numbers (the vertical ones, on the left of
a BookMapRow) on the baseline of the BookMapRow, where
page slots rise from.
Also tweak the page number spike below the baseline used
in PageBrowser.
Enabled by default, to get the current behaviour.
Disabling it allows people with small fingers to jump
directly to that page in Reader.
Allows hairy-boxes amateurs to get small markers (under
the baseline) every 10 pages (with -/+ to get small or
medium markers, with optionally smaller ones every 5 pages).
Add a restricted but convenient mode showing BookMap like
in initial view, while still allowing chapter levels to be
tweaked. This allows getting back to this view with another
gesture to see the overall progress in the book, while
still having the normal BookMap in flat mode acting as
an alternative ToC.
Available as an action to associate to a gesture, and
with long-press on the "Book map" menu item.
@poire-z poire-z added this to the 2023.06 milestone Jun 2, 2023
})
if self.overview_mode then
text = text .. "\n\n" .. _([[
In overview mode, the book map is always in grid mode and ensured to fit on a single screen. Chapter levels can be changed for the most confortable overview.]])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Frenzie : no comment about this added sentence wording ?
(The rest is just splitted previous text :/ for reuse in the 2 cases.)

Copy link
Member

Choose a reason for hiding this comment

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

Actually I hadn't noticed that there was some added. There's a French typo in comfortable.

And maybe "made to fit" would sound a touch better than "ensured to fit"?

@poire-z poire-z merged commit 04425b7 into koreader:master Jun 4, 2023
@poire-z poire-z deleted the bookmap_enhancements2 branch June 4, 2023 22:58
Frenzie pushed a commit that referenced this pull request Jun 5, 2023
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.

FR: BookMap enhancements
2 participants