summaryrefslogtreecommitdiff
path: root/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration
diff options
context:
space:
mode:
authorJakob Heher <jakob.heher@iaik.tugraz.at>2022-08-03 14:43:06 +0200
committerJakob Heher <jakob.heher@iaik.tugraz.at>2022-08-03 14:43:06 +0200
commited3ea068ac80cc3a9e9a4b07599c82d732320017 (patch)
tree89d55f047c167c922eea2f5b66413bf41d9083bd /pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration
parent8ed8c7ff59cf052aaab1fbe07deca55aa1dcacc0 (diff)
downloadpdf-over-ed3ea068ac80cc3a9e9a4b07599c82d732320017.tar.gz
pdf-over-ed3ea068ac80cc3a9e9a4b07599c82d732320017.tar.bz2
pdf-over-ed3ea068ac80cc3a9e9a4b07599c82d732320017.zip
don't pointlessly redraw signature preview unless it's actually needed
Diffstat (limited to 'pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration')
-rw-r--r--pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration/SimpleConfigurationComposite.java106
1 files changed, 49 insertions, 57 deletions
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration/SimpleConfigurationComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration/SimpleConfigurationComposite.java
index 34155668..e36164f9 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration/SimpleConfigurationComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/configuration/SimpleConfigurationComposite.java
@@ -45,7 +45,6 @@ import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Canvas;
import org.eclipse.swt.widgets.Combo;
import org.eclipse.swt.widgets.Composite;
-import org.eclipse.swt.widgets.Control;
import org.eclipse.swt.widgets.FileDialog;
import org.eclipse.swt.widgets.Group;
import org.eclipse.swt.widgets.Label;
@@ -53,7 +52,6 @@ import org.eclipse.swt.widgets.Text;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import at.asit.pdfover.Util;
import at.asit.pdfover.commons.Constants;
import at.asit.pdfover.commons.Messages;
import at.asit.pdfover.commons.Profile;
@@ -98,8 +96,6 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
protected final Combo cmbSignatureLang;
protected String logoFile = null;
- protected Image sigPreview = null;
- protected Image logo = null;
protected final Group grpSignatureProfile;
protected final Combo cmbSignatureProfiles;
@@ -195,7 +191,7 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
this.cSigPreview = new Canvas(containerComposite, SWT.RESIZE);
StateComposite.anchor(cSigPreview).left(0, 5).right(100,-5).top(btnBrowseLogo,5).bottom(100,-5).set();
StateComposite.setFontHeight(cSigPreview, Constants.TEXT_SIZE_NORMAL);
- this.cSigPreview.addPaintListener(e -> imagePaintControl(e, SimpleConfigurationComposite.this.sigPreview));
+ this.cSigPreview.addPaintListener(e -> SimpleConfigurationComposite.this.paintSignaturePreview(e));
DropTarget dnd_target = new DropTarget(containerComposite, DND.DROP_DEFAULT | DND.DROP_COPY);
final FileTransfer fileTransfer = FileTransfer.getInstance();
@@ -345,14 +341,49 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
reloadResources();
}
- static void imagePaintControl(PaintEvent e, Image i) {
- if (i == null)
- return;
+ boolean needSigPreviewUpdate = true;
+ private Image sigPreview = null;
+ void paintSignaturePreview(PaintEvent evt) {
+ if (needSigPreviewUpdate)
+ {
+ log.info("updating signature preview");
+ String image = this.configurationContainer.getEmblemPath();
+ ImageData img = null;
+
+ try {
+ PdfAs4SignatureParameter param = new PdfAs4SignatureParameter();
+ param.signatureProfileName = this.configurationContainer.getSignatureProfile().name();
+ if(this.configurationContainer.signatureNote != null && !this.configurationContainer.signatureNote.isEmpty()) {
+ param.setProperty("SIG_NOTE", this.configurationContainer.signatureNote);
+ }
+
+ param.signatureLanguage = this.configurationContainer.signatureLocale.getLanguage();
+ param.enablePDFACompat = this.configurationContainer.signaturePDFACompat;
+ if (image != null && !image.trim().isEmpty()) {
+ param.emblem = new Emblem(image);
+ }
+ // TODO this is super slow, can we get rid of it?
+ // SWT_AWT, same as PositioningComposite, maybe?
+ // .... i hate this
+ img = ImageConverter.convertToSWT((BufferedImage) param.getPlaceholder());
+ } catch (Exception e) {
+ log.error("Failed to load image for display...", e);
+ }
+
+ if (img != null) {
+ this.sigPreview = new Image(this.getDisplay(), img);
+ } else {
+ this.sigPreview = null;
+ }
+ needSigPreviewUpdate = false;
+ }
- Rectangle r = i.getBounds();
+ if (this.sigPreview == null)
+ return;
+ Rectangle r = this.sigPreview.getBounds();
int srcW = r.width;
int srcH = r.height;
- Point p = ((Control) e.widget).getSize();
+ Point p = ((Canvas)evt.widget).getSize();
float dstW = p.x;
float dstH = p.y;
@@ -365,7 +396,7 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
int x = (int) ((dstW / 2) - (w / 2));
int y = (int) ((dstH / 2) - (h / 2));
- e.gc.drawImage(i, 0, 0, srcW, srcH, x, y, (int) w, (int) h);
+ evt.gc.drawImage(this.sigPreview, 0, 0, srcW, srcH, x, y, (int) w, (int) h);
}
/**
@@ -412,51 +443,12 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
this.configurationContainer.setEmblem(filename);
this.btnClearImage.setVisible(filename != null);
- this.updateSignatureBlockPreview();
+ this.signatureBlockPreviewChanged();
this.doLayout();
}
- void updateSignatureBlockPreview() {
- String image = this.configurationContainer.getEmblemPath();
- ImageData img = null;
- ImageData logo = null;
-
- try {
- PdfAs4SignatureParameter param = new PdfAs4SignatureParameter();
- param.signatureProfileName = this.configurationContainer.getSignatureProfile().name();
- if(this.configurationContainer.signatureNote != null && !this.configurationContainer.signatureNote.isEmpty()) {
- param.setProperty("SIG_NOTE", this.configurationContainer.signatureNote);
- }
-
- param.signatureLanguage = this.configurationContainer.signatureLocale.getLanguage();
- param.enablePDFACompat = this.configurationContainer.signaturePDFACompat;
- if (image != null && !image.trim().isEmpty()) {
- logo = new ImageData(image);
- param.emblem = new Emblem(image);
- }
- img = ImageConverter.convertToSWT((BufferedImage) param.getPlaceholder());
- } catch (Exception e) {
- log.error("Failed to load image for display...", e);
- }
-
- if (img != null) {
- this.sigPreview = new Image(this.getDisplay(), img);
- } else {
- this.sigPreview = null;
- }
-
- if (logo != null) {
- try {
- File imgFile = new File(image);
- this.logo = new Image(this.getDisplay(),
- ImageConverter.convertToSWT(Util.readImageWithEXIFRotation(imgFile)));
- } catch (IOException e) {
- log.error("Error reading image", e);
- }
- } else {
- this.logo = null;
- }
-
+ void signatureBlockPreviewChanged() {
+ needSigPreviewUpdate = true;
this.cSigPreview.redraw();
}
@@ -506,7 +498,7 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
if ((previous != null) && (txtSignatureNote.getText().equals(getDefaultSignatureBlockNoteTextFor(null, previous))))
txtSignatureNote.setText(getDefaultSignatureBlockNoteTextFor(null, selected));
- updateSignatureBlockPreview();
+ signatureBlockPreviewChanged();
}
@@ -523,7 +515,7 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
if (txtSignatureNote.getText().equals(getDefaultSignatureBlockNoteTextFor(oldProfile, null)))
txtSignatureNote.setText(getDefaultSignatureBlockNoteTextFor(newProfile, null));
- updateSignatureBlockPreview();
+ signatureBlockPreviewChanged();
}
String getDefaultSignatureBlockNoteTextFor(Profile profile, Locale locale){
@@ -548,7 +540,7 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
void processSignatureNoteChanged() {
String note = this.txtSignatureNote.getText();
this.configurationContainer.signatureNote = note;
- updateSignatureBlockPreview();
+ signatureBlockPreviewChanged();
}
/*
@@ -609,7 +601,7 @@ public class SimpleConfigurationComposite extends ConfigurationCompositeBase {
this.txtSignatureNote.setText(note);
}
- this.updateSignatureBlockPreview();
+ this.signatureBlockPreviewChanged();
this.performSignatureLangSelectionChanged(this.configurationContainer.signatureLocale, null);