Skip to content

KoptInterface: fix panel zooming when dewatermark is on#15067

Merged
Frenzie merged 2 commits into
koreader:masterfrom
Arcohol:fix-panel-zooming
Mar 3, 2026
Merged

KoptInterface: fix panel zooming when dewatermark is on#15067
Frenzie merged 2 commits into
koreader:masterfrom
Arcohol:fix-panel-zooming

Conversation

@Arcohol

@Arcohol Arcohol commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Currently, panel zoom does not work when dewatermark is on.
This PR fixes the issue by calculating and using the correct bbox. part_tile is not cached yet; I'm not sure if caching is necessary.
Tested on Kindle Paperwhite (12th Generation).


This change is Reviewable

@Frenzie

Frenzie commented Mar 3, 2026

Copy link
Copy Markdown
Member

part_tile is not cached yet; I'm not sure if caching is necessary.

Are you referring to not caching some intermediate step or the end result?

@Arcohol

Arcohol commented Mar 3, 2026

Copy link
Copy Markdown
Contributor Author

renderOptimizedPage returns either a full-page tile or a partial tile (the zoomed-in panel). Currently, only the full-page tile is added to the cache (this is the original behavior). This is mainly because I’m new to the project and don’t yet fully understand how the caching system works or why it was designed this way. I don't want to introduce any breaking changes.

@Frenzie Frenzie changed the title KoptInterface: fix panel zooming KoptInterface: fix panel zooming when dewatermark is on Mar 3, 2026
Comment thread frontend/document/koptinterface.lua Outdated
@Frenzie Frenzie merged commit f12ff96 into koreader:master Mar 3, 2026
2 checks passed
@Frenzie Frenzie added this to the 2026.02 milestone Mar 3, 2026
Frenzie pushed a commit that referenced this pull request Mar 17, 2026
)

Due to changes to renderOptimizedPage(...) made by #15067, which was merged after #14954 was made, a regression was introduced that causes instability and even crashes on mobile devices and e-readers when Invert Document is enabled along with Reflow or Auto Straighten.

This PR fixes the regression and simplifies the drawContextPage(...) method.
jqs7 pushed a commit to jqs7/koreader that referenced this pull request Mar 19, 2026
…eader#15123)

Due to changes to renderOptimizedPage(...) made by koreader#15067, which was merged after koreader#14954 was made, a regression was introduced that causes instability and even crashes on mobile devices and e-readers when Invert Document is enabled along with Reflow or Auto Straighten.

This PR fixes the regression and simplifies the drawContextPage(...) method.
@Arcohol Arcohol deleted the fix-panel-zooming branch March 22, 2026 14:14
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
…eader#15123)

Due to changes to renderOptimizedPage(...) made by koreader#15067, which was merged after koreader#14954 was made, a regression was introduced that causes instability and even crashes on mobile devices and e-readers when Invert Document is enabled along with Reflow or Auto Straighten.

This PR fixes the regression and simplifies the drawContextPage(...) method.
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.

2 participants