Merge "Issue #1229 Changes for non-blocking ImagingDialog." into development

Former-commit-id: de3314a441 [formerly 77de744b4e] [formerly de3314a441 [formerly 77de744b4e] [formerly 74a9c05db5 [formerly 2a7f9d1d5bab1c3edc8c6ac2f424e0b8865bc4c6]]]
Former-commit-id: 74a9c05db5
Former-commit-id: bc0ef0922e [formerly 3b3c636de4]
Former-commit-id: a26e49ad74
This commit is contained in:
Lee Venable 2012-10-17 14:07:37 -05:00 committed by Gerrit Code Review
commit 09428e1308
3 changed files with 21 additions and 10 deletions

View file

@ -37,6 +37,7 @@ import com.raytheon.viz.ui.dialogs.ImagingDialog;
* Date Ticket# Engineer Description * Date Ticket# Engineer Description
* ------------ ---------- ----------- -------------------------- * ------------ ---------- ----------- --------------------------
* Nov 22, 2006 chammack Initial Creation. * Nov 22, 2006 chammack Initial Creation.
* Oct 17, 2012 1229 rferrel Changes for non-blocking ImagingDialog.
* *
* </pre> * </pre>
* *
@ -56,7 +57,7 @@ public class ImagePropertiesAction extends AbstractHandler {
*/ */
@Override @Override
public Object execute(ExecutionEvent arg0) throws ExecutionException { public Object execute(ExecutionEvent arg0) throws ExecutionException {
if (dialog == null) { if (dialog == null || dialog.getShell() == null || dialog.isDisposed()) {
IDisplayPaneContainer container = EditorUtil IDisplayPaneContainer container = EditorUtil
.getActiveVizContainer(); .getActiveVizContainer();
if (container != null) { if (container != null) {
@ -64,11 +65,12 @@ public class ImagePropertiesAction extends AbstractHandler {
.getCurrentWindow().getShell(), container); .getCurrentWindow().getShell(), container);
// initalize // initalize
dialog.open(); dialog.open();
} else {
dialog = null; dialog = null;
} }
} else { } else {
dialog.refreshComponents(); dialog.refreshComponents();
dialog.open(); dialog.bringToTop();
} }
return null; return null;

View file

@ -32,6 +32,7 @@ import com.raytheon.viz.ui.dialogs.ImagingDialog;
* Date Ticket# Engineer Description * Date Ticket# Engineer Description
* ------------ ---------- ----------- -------------------------- * ------------ ---------- ----------- --------------------------
* Nov 22, 2006 chammack Initial Creation. * Nov 22, 2006 chammack Initial Creation.
* Oct 17, 2012 1229 rferrel Changes for non-blocking ImagingDialog.
* *
* </pre> * </pre>
* *
@ -39,6 +40,7 @@ import com.raytheon.viz.ui.dialogs.ImagingDialog;
* @version 1 * @version 1
*/ */
public class ImagingAction extends AbstractRightClickAction { public class ImagingAction extends AbstractRightClickAction {
private ImagingDialog dialog;
/* /*
* (non-Javadoc) * (non-Javadoc)
@ -47,10 +49,14 @@ public class ImagingAction extends AbstractRightClickAction {
*/ */
@Override @Override
public void run() { public void run() {
AbstractVizResource<?, ?> rsc = getTopMostSelectedResource(); if (dialog == null || dialog.getShell() == null || dialog.isDisposed()) {
ImagingDialog id = new ImagingDialog(VizWorkbenchManager.getInstance() AbstractVizResource<?, ?> rsc = getTopMostSelectedResource();
.getCurrentWindow().getShell(), rsc); dialog = new ImagingDialog(VizWorkbenchManager.getInstance()
id.open(); .getCurrentWindow().getShell(), rsc);
dialog.open();
} else {
dialog.bringToTop();
}
} }
/* /*

View file

@ -83,6 +83,7 @@ import com.raytheon.viz.ui.editor.IMultiPaneEditor;
* show full colormap names, and added comments &amp; Javadoc. * show full colormap names, and added comments &amp; Javadoc.
* Aug 20, 2008 dglazesk Updated for the new ColorMap interface * Aug 20, 2008 dglazesk Updated for the new ColorMap interface
* Feb 10, 2011 #7842 bkowal set caveStyle for dialog to INDEPENDENT SHELL * Feb 10, 2011 #7842 bkowal set caveStyle for dialog to INDEPENDENT SHELL
* Oct 17, 2012 #1229 rferrel Make dialog non-blocking.
* *
* </pre> * </pre>
* *
@ -92,7 +93,7 @@ import com.raytheon.viz.ui.editor.IMultiPaneEditor;
public class ImagingDialog extends CaveSWTDialog implements public class ImagingDialog extends CaveSWTDialog implements
IVizEditorChangedListener, IRenderableDisplayChangedListener, IVizEditorChangedListener, IRenderableDisplayChangedListener,
AddListener, RemoveListener, IResourceDataChanged { AddListener, RemoveListener, IResourceDataChanged {
private static final transient IUFStatusHandler statusHandler = UFStatus private final transient IUFStatusHandler statusHandler = UFStatus
.getHandler(ImagingDialog.class); .getHandler(ImagingDialog.class);
private static enum Type { private static enum Type {
@ -149,7 +150,7 @@ public class ImagingDialog extends CaveSWTDialog implements
*/ */
private Button combineNextImage; private Button combineNextImage;
private static final String UNSAVED_CMAP_DISPLAY_NAME = "Untitled Colormap"; private final String UNSAVED_CMAP_DISPLAY_NAME = "Untitled Colormap";
private boolean fromControl = false; private boolean fromControl = false;
@ -196,7 +197,8 @@ public class ImagingDialog extends CaveSWTDialog implements
* @param dialogTitle * @param dialogTitle
*/ */
public ImagingDialog(Shell parentShell, IDisplayPaneContainer initialEditor) { public ImagingDialog(Shell parentShell, IDisplayPaneContainer initialEditor) {
super(parentShell, SWT.DIALOG_TRIM | SWT.MIN, CAVE.INDEPENDENT_SHELL); super(parentShell, SWT.DIALOG_TRIM | SWT.MIN, CAVE.INDEPENDENT_SHELL
| CAVE.DO_NOT_BLOCK);
setText("Imaging..."); setText("Imaging...");
this.currentEditor = initialEditor; this.currentEditor = initialEditor;
@ -207,7 +209,8 @@ public class ImagingDialog extends CaveSWTDialog implements
} }
public ImagingDialog(Shell parentShell, AbstractVizResource<?, ?> rscToEdit) { public ImagingDialog(Shell parentShell, AbstractVizResource<?, ?> rscToEdit) {
super(parentShell, SWT.DIALOG_TRIM | SWT.MIN, CAVE.INDEPENDENT_SHELL); super(parentShell, SWT.DIALOG_TRIM | SWT.MIN, CAVE.INDEPENDENT_SHELL
| CAVE.DO_NOT_BLOCK);
setText("Imaging..."); setText("Imaging...");
this.rscToEdit = rscToEdit; this.rscToEdit = rscToEdit;
} }