marcus/pdf-umbenenner
1
0
Fork 0
Code Issues 14 Releases 2 Activity

Korrigiere Reihenfolge in resetToFitView für zuverlässige Zentrierung

Browse Source 
Zwei zusammenwirkende Ursachen für die linksbündige Anzeige nach Zoom-Reset:

1. Die Property-Bindungen wurden vor setFitToWidth(true) gesetzt. Zu diesem
   Zeitpunkt sizet der viewStack noch nach der zoom-großen ImageView, sodass
   die Bindungen die imageView an die Zoom-Breite gekoppelt haben statt an
   die Viewport-Breite.

2. Verbleibende H/V-Werte aus Pan-/Zoom-Modus (insbesondere hvalue=0.0 nach
   Pan zum linken Rand) wurden nicht zurückgesetzt. Bei minimalsten
   Rounding-/Border-Differenzen wirkt hvalue auch im fit-aktiven Modus und
   richtet den Content links bündig aus.

Fix: setFitToWidth/Height(true) sofort; Bindings und setHvalue(0.5)/
setVvalue(0.5) im Platform.runLater nach abgeschlossenem Layout-Pass.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcus van Elst
2026-05-05 15:26:31 +02:00
parent a8d8a4a3c1
commit 3fb511601c
1 changed files with 25 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/PdfPreviewPane.java
+25 -9
View File
@@ -743,13 +743,20 @@ public final class PdfPreviewPane {
* Setzt Zoom, Akkumulator und Pan-Zustand zurück und reaktiviert den Fit-to-View-Modus.
* Wird beim Laden einer neuen Datei und beim Leeren der Komponente aufgerufen.
*
* <p>Die Property-Bindungen von {@code fitWidth}/{@code fitHeight} an
* {@code viewStack.widthProperty()}/{@code viewStack.heightProperty()} werden
* wiederhergestellt – damit ist der ImageView-Zustand identisch zum initialen
* Zustand aus dem Konstruktor. Der {@link StackPane} zentriert den ImageView
* dann automatisch über die zuvor gesetzte {@code Pos.CENTER}-Ausrichtung;
* eine explizite H/V-Wert-Manipulation am ScrollPane ist im fit-Modus
* wirkungslos und daher nicht nötig.
* <p>Reihenfolge der Aktionen ist kritisch:
* <ol>
* <li>{@code setFitToWidth(true)} und {@code setFitToHeight(true)} sofort,
* damit der nächste Layout-Pass den {@code viewStack} auf Viewport-Größe
* zurückrechnet.</li>
* <li>Property-Bindungen und H/V-Reset im {@code Platform.runLater}, damit
* sie auf die bereits zurückgerechneten {@code viewStack}-Dimensionen
* wirken und nicht auf die noch zoom-große Breite.</li>
* </ol>
* Ohne diese Reihenfolge würden die Bindungen die imageView kurz an die
* Zoom-Größe koppeln, und ein verbleibender H/V-Wert aus dem Pan-/Zoom-Modus
* (z. B. {@code hvalue=0.0} nach Pan zum linken Rand) würde die PDF wegen
* kleinster Rounding-/Border-Differenzen links/oben bündig statt zentriert
* anzeigen, obwohl der ScrollPane fit-aktiv ist.
*/
private void resetToFitView() {
zoomLevel = 1.0;
@@ -760,10 +767,19 @@ public final class PdfPreviewPane {
panStartY = -1;
viewStack.setCursor(null);
if (!scrollPane.isFitToWidth()) {
imageView.fitWidthProperty().bind(viewStack.widthProperty());
imageView.fitHeightProperty().bind(viewStack.heightProperty());
// 1. ScrollPane in Fit-Modus schalten, damit der nächste Layout-Pass
// den viewStack auf Viewport-Größe zurückrechnet
scrollPane.setFitToWidth(true);
scrollPane.setFitToHeight(true);
// 2./3. Bindings und H/V-Reset erst nach abgeschlossenem Layout-Pass,
// damit sie auf die zurückgerechneten Dimensionen wirken und
// Rounding-Reste aus Pan/Zoom die Zentrierung nicht verschieben
Platform.runLater(() -> {
imageView.fitWidthProperty().bind(viewStack.widthProperty());
imageView.fitHeightProperty().bind(viewStack.heightProperty());
scrollPane.setHvalue(0.5);
scrollPane.setVvalue(0.5);
});
}
}