Skip to content

feat: allow user patches on non-paging documents in paintTo#71

Open
OGKevin wants to merge 1 commit into
mainfrom
khellemun/push/nzznrmyokmpz
Open

feat: allow user patches on non-paging documents in paintTo#71
OGKevin wants to merge 1 commit into
mainfrom
khellemun/push/nzznrmyokmpz

Conversation

@OGKevin
Copy link
Copy Markdown
Member

@OGKevin OGKevin commented Dec 30, 2025

Refactored ReaderView:paintTo to delegate to the original method for non-paging documents, enabling user patches (such as header/footer patches) to work correctly. Simplified paging logic and improved dithering handling for both paging and non-paging documents.

  • Save and use the original paintTo for non-paging documents
  • Only override paintTo for paging documents (PDFs, CBZs, etc.)
  • Simplify page drawing logic for clarity and maintainability
  • Adjust dithering logic to respect user settings on paging documents

Fixes: #65
Change-Id: c00c8d1bfda0698d663a6045b0930448
Change-Id-Short: nzznrmyokmpz

@OGKevin OGKevin force-pushed the khellemun/push/nzznrmyokmpz branch 2 times, most recently from 13f3017 to 7b8cf8f Compare December 30, 2025 15:44
Refactored ReaderView:paintTo to delegate to the original method for
non-paging documents, enabling user patches (such as header/footer
patches) to work correctly. Simplified paging logic and improved
dithering handling for both paging and non-paging documents.

- Save and use the original paintTo for non-paging documents
- Only override paintTo for paging documents (PDFs, CBZs, etc.)
- Simplify page drawing logic for clarity and maintainability
- Adjust dithering logic to respect user settings on paging documents

Fixes: #65
Change-Id: c00c8d1bfda0698d663a6045b0930448
Change-Id-Short: nzznrmyokmpz
@OGKevin OGKevin force-pushed the khellemun/push/nzznrmyokmpz branch from 7b8cf8f to e5abfba Compare December 30, 2025 15:45
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.

Conflict with Header and Footer Patches

1 participant