[RobinD42]

Pull request

Contributor checklist

• The change is scoped to the related issue or feature discussion.
• Tests and manual-validation status are documented.
• No credentials, cookies, API keys, private server URLs, or generated local data are included.

Summary

Add per-library Show header and Show header/footer borders options for EPUB, PDF, and comic readers. Both are off by default. The optional header places the title on the left and author on the right, truncating the title only when it would overlap the author. Thin, screen-wide borders optionally appear below the header and above an enabled progress footer.

Header positioning uses the viewport’s on-screen location, status-bar inset, and visible display frame to avoid status-bar overlap across devices. Preference and positioning regression tests, translated settings strings, and the UI/UX documentation are included.

Verification

• .\gradlew.bat :app:compileDebugKotlin :app:testDebugUnitTest :app:lintDebug :app:assembleDebug — passed.
• git diff --check — passed.
• Manual validation on 3 physical devices with and without cut outs, plus the emulator. I iterated with the AI until it worked correctly in all cases I was testing, with and without the status bar.

NOTE: It appears that for PDF and Comics that the page view always starts at the top of the screen, regardless of whether the status bar (and now the header) is hidden or not. I expect it's the same with EPUB but the larger default top margin prevents that from being obvious. I didn't try changing it in this PR in case there is a good reason for it to be this way. I can attempt it another time if you want. IMO, it would be nice if the top of the page behaved more like the bottom does with the navbar and/or progress info.

BTW: Feel free to change things in my PRs if you want, or to request changes.

Screenshots

[van-geaux]

@RobinD42, I noticed readerContainer reserves space for the footer but not the header, while headerView is added as an overlay. Since PDF and comic pages start at the top, the header may cover page content when enabled. Could you confirm whether you see this on your devices too? If so, please reserve space for the header while keeping the current layout when it’s off. Thanks!

[RobinD42]

Okay, I'll work on that when I switch back to the branch.

[van-geaux]

@RobinD42, thanks for the update. I reviewed the latest changes and found no blocking issues. The PR looks ready to merge. Thanks for your work on this!