diff options
author | Jakob Heher <jakob.heher@iaik.tugraz.at> | 2022-11-03 15:01:23 +0100 |
---|---|---|
committer | Jakob Heher <jakob.heher@iaik.tugraz.at> | 2022-11-03 15:01:23 +0100 |
commit | d0ba2d54ed60c67fb2ca91f1eb534d800e3d7e6c (patch) | |
tree | 3b8dec28497eb8dd6c080c3dd21b30415a35489c /pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java | |
parent | 1835bf6f84516b74bbd6334dda56a9f3953c5590 (diff) | |
download | pdf-over-d0ba2d54ed60c67fb2ca91f1eb534d800e3d7e6c.tar.gz pdf-over-d0ba2d54ed60c67fb2ca91f1eb534d800e3d7e6c.tar.bz2 pdf-over-d0ba2d54ed60c67fb2ca91f1eb534d800e3d7e6c.zip |
fix UX race condition in positioningpanel
cf. #129
Diffstat (limited to 'pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java')
-rw-r--r-- | pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java index 85b950a6..5be8a4c7 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java @@ -26,6 +26,7 @@ import java.awt.event.KeyEvent; import java.awt.event.KeyListener; import java.awt.event.MouseWheelEvent; import java.awt.event.MouseWheelListener; +import java.awt.geom.Point2D; import org.apache.pdfbox.pdmodel.PDDocument; import org.eclipse.swt.SWT; @@ -123,6 +124,8 @@ public class PositioningComposite extends StateComposite { requestFocus(); } + @Override public void onDisplay() { if (this.viewer.getSigPagePos() == null) this.btnSign.setEnabled(false); } + /** * Set the PDF Document to display * @@ -132,6 +135,7 @@ public class PositioningComposite extends StateComposite { public void displayDocument(final PDDocument document) { EventQueue.invokeLater(() -> { this.viewer.setDocument(document); + getDisplay().asyncExec(() -> this.btnSign.setEnabled(true)); }); if (document != null) @@ -333,9 +337,12 @@ public class PositioningComposite extends StateComposite { if (this.currentPage == 0) { this.position = new SignaturePosition(); } else { + Point2D pos = this.viewer.getSigPagePos(); + if (pos == null) return; + this.position = new SignaturePosition( - this.viewer.getSigPageX(), - this.viewer.getSigPageY(), + pos.getX(), + pos.getY(), this.currentPage); } PositioningComposite.this.state.updateStateMachine(); |