Merge pull request #905 from Mailaender/target-label-refresh

Fixed Target Label Settings not refreshing
eclipse · Feb 17, 2022 · 26f92e5 · 26f92e5
2 parents 7c7fac6 + ac91c67
commit 26f92e5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/....xxd.ui/src/org/eclipse/chemclipse/ux/extension/xxd/ui/actions/TargetLabelEditAction.java b/....xxd.ui/src/org/eclipse/chemclipse/ux/extension/xxd/ui/actions/TargetLabelEditAction.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2020 Lablicate GmbH.
+ * Copyright (c) 2020, 2022 Lablicate GmbH.
  *
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
@@ -152,6 +152,8 @@ public void setShowPreview(boolean preview) {
 			baseChart.getPlotArea().removeCustomPaintListener(targetReferenceLabelMarker);
 			settingsWizardListener.setPreviewSettings(null);
 			if(settingsChanged) {
+				labelEditSettings.getChromatogramUI().update();
+				chromatogramChart.getBaseChart().redraw();
 				listeners.forEach(Runnable::run);
 			}
 		}

diff --git a/...i/src/org/eclipse/chemclipse/ux/extension/xxd/ui/wizards/TargetDisplaySettingsWizard.java b/...i/src/org/eclipse/chemclipse/ux/extension/xxd/ui/wizards/TargetDisplaySettingsWizard.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2019, 2020 Lablicate GmbH.
+ * Copyright (c) 2019, 2022 Lablicate GmbH.
  *
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
@@ -46,11 +46,11 @@ protected void createButtonsForButtonBar(Composite parent) {
 		 * Initial width and height.
 		 */
 		wizardDialog.setPageSize(DEFAULT_WIDTH, DEFAULT_HEIGHT);
-		//
-		if(wizardDialog.open() == Window.OK) {
-			return true;
-		} else {
-			return false;
-		}
+		return (wizardDialog.open() == Window.OK);
+	}
+
+	private TargetDisplaySettingsWizard() {
+		// static access only
+
 	}
 }