Repository navigation
fix(viewer): voorkom miniatuurflits bij paginawissels (#854) - #857
Closed
oscar981-op wants to merge 1 commit into
Closed
oscar981-op wants to merge 1 commit into
oscar981-op wants to merge 1 commit into
Conversation
Merged
Collaborator
|
Thank you @oscar981-op! This is released in 2026.44.4 as one squashed commit with you as the author (merged through #862). Review fix on top: a zoom snapshot is also dropped when only the view rotation changes, so it is never drawn in the old orientation. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Integrated with maintainer review fixes via #862 (upstream main cdda6a0). The original tested branch is retained as reference; do not merge it again.
Closes #854.
Bij bladeren in Single Page op bijvoorbeeld 300% zoom verscheen eerst een wazige, passend gemaakte zijbalkminiatuur boven het vaste viewportcanvas. De echte pagina verscheen daarna op de bestaande zoom en pan. De aparte overlay en zijn timers zijn verwijderd: navigatie toont de normale pagina-render zonder deze tussenstap.
setPage()herstelt nu ook bij paginawissels binnen hetzelfde document direct de beste beschikbare bitmap uit de bestaande cache, met dezelfde rotatiesleutel en 4096-pixelgrens als de orchestrator. Een zoomsnapshot van de vertrekkende pagina wordt meteen losgelaten, zodat die de nieuwe pagina niet alsnog kan bedekken.Validatie:
goToPage(): een vertraagde eerste navigatie en vijf pagina-overgangen blijven op 300%; MutationObserver detecteert geen thumbnail-overlay, ook niet binnen één frame.git diff --checkgeslaagd.30a2806557babe032cdbb1eb116a15bf840cee7a; conceptrelease aangemaakt met dezelfde commit als target. Er wordt geen release gepubliceerd.Afweging: deze kleine fix verwijdert de tijdelijke zijbalkminiatuur volledig. Een nog niet gecachte pagina wacht op de gewone render en kan daarbij kort een leeg paginavlak tonen; er is geen nieuwe thumbnail-decoder of vertraagde stand-in. Een lager-resolutie cachetreffer kan nog zacht zijn totdat de scherpe render arriveert, maar gebruikt dezelfde paginageometrie. Native Windows-interactie is hier niet handmatig getest; viewportgedrag is met canvas/cache-stubs getest, browsernavigatie met echte PDF.js. Conceptstatus blijft behouden zolang installercontroles of verplichte review ontbreken; het account heeft geen upstream merge-recht.
Installerresultaat op head
30a2806557babe032cdbb1eb116a15bf840cee7a: alle vijf upstream CI-checks geslaagd. Webbundel, Android en MCPB geslaagd; Windows-jobs stranden op de ontbrekende Azure-signing-endpoint, Linux op het decoderen van de updater-sleutel/wachtwoord; macOS geannuleerd. Conceptrelease blijft ongepubliceerd. Conceptstatus behouden wegens installerblokkades en ontbrekende verplichte review; het account heeft geen upstream merge-recht.