Skip to content

fix: correct home cover dimensions#552

Closed
pablohc wants to merge 4 commits intocrosspoint-reader:masterfrom
pablohc:fix/home-cover
Closed

fix: correct home cover dimensions#552
pablohc wants to merge 4 commits intocrosspoint-reader:masterfrom
pablohc:fix/home-cover

Conversation

@pablohc
Copy link
Contributor

@pablohc pablohc commented Jan 26, 2026

Summary

  • What is the goal of this PR?
    Fixes the cover image dimensions for the home screen (Continue Reading card).

  • What changes are included?

    • Modifies image generation to use 400px height with proportional width
    • Optimizes rendering by removing unnecessary centering calculations
    • Updates Epub and Xtc classes to use the new fixed dimension system

Additional Context

Performance: Images use 1-bit format for faster rendering, but loading times still need optimization.


AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing, please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? < YES >

nkocher added a commit to nkocher/crosspoint-reader that referenced this pull request Jan 31, 2026
crosspoint-reader#576, crosspoint-reader#587, crosspoint-reader#600 into custom-v16

Integrate 5 upstream PRs:
- crosspoint-reader#612: WiFi error text fixes
- crosspoint-reader#552: Home cover dimension fix
- crosspoint-reader#576: Don't wake on USB connect
- crosspoint-reader#587: Boot loop escape
- crosspoint-reader#600: System-wide continuous navigation (ButtonNavigator)

Manually added ButtonNavigator to EpubReaderMenuActivity (custom file
not in upstream) for hold-to-navigate support in reading menu.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@pablohc pablohc closed this Feb 3, 2026
@pablohc
Copy link
Contributor Author

pablohc commented Feb 3, 2026

Replaced by: #683

@pablohc pablohc deleted the fix/home-cover branch February 3, 2026 16:05
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