From 68f61a9454ff0e64ffbcb8c646015c3378f2bb95 Mon Sep 17 00:00:00 2001
From: Mark Peters <Mark.A.Peters@raytheon.com>
Date: Mon, 11 Aug 2014 10:08:38 -0500
Subject: [PATCH] Omaha #3472 Clicking & dragging the Az/Ran Overlay tool off
 the map causes the map to jump

Change-Id: I5dc3fc528a8e62dd32d87524f99adb1a8c9e610e

Former-commit-id: 490f7ea209115cb66a13fe03d94f8b31a4a6a863 [formerly a73ca1f8c1df176d6a6b0040d701e90512950b7d [formerly 1b0ac4736f7d63c5e49a48a28473ad6a38e6b6ea]]
Former-commit-id: a73ca1f8c1df176d6a6b0040d701e90512950b7d
Former-commit-id: 5121ddaa125fba3b135be498b7eba77e40d151b8
---
 .../awipstools/ui/layer/AzimuthToolLayer.java | 105 ++++++++++++++----
 1 file changed, 82 insertions(+), 23 deletions(-)

diff --git a/cave/com.raytheon.viz.awipstools/src/com/raytheon/viz/awipstools/ui/layer/AzimuthToolLayer.java b/cave/com.raytheon.viz.awipstools/src/com/raytheon/viz/awipstools/ui/layer/AzimuthToolLayer.java
index 91b37956cc..3f9bce2f55 100644
--- a/cave/com.raytheon.viz.awipstools/src/com/raytheon/viz/awipstools/ui/layer/AzimuthToolLayer.java
+++ b/cave/com.raytheon.viz.awipstools/src/com/raytheon/viz/awipstools/ui/layer/AzimuthToolLayer.java
@@ -59,8 +59,8 @@ import com.raytheon.uf.viz.core.rsc.capabilities.OutlineCapability;
 import com.raytheon.uf.viz.core.rsc.tools.AbstractMovableToolLayer;
 import com.raytheon.uf.viz.core.rsc.tools.GenericToolsResourceData;
 import com.raytheon.uf.viz.points.PointsDataManager;
+import com.raytheon.viz.awipstools.common.ToolsUiUtil;
 import com.raytheon.viz.awipstools.common.stormtrack.StormTrackDisplay;
-import com.raytheon.viz.awipstools.common.stormtrack.StormTrackUIManager;
 import com.raytheon.viz.ui.cmenu.IContextMenuContributor;
 import com.raytheon.viz.ui.input.EditableManager;
 import com.raytheon.viz.ui.input.InputAdapter;
@@ -90,6 +90,10 @@ import com.vividsolutions.jts.geom.Coordinate;
  *  07-28-14    #3430        mapeters    Updated the 'handleMouseUp' function to prevent
  *                                       errors created when MB3 clicking off the map
  *                                       in editable mode.
+ *  08-11-14    #3472        mapeters    Added Mode enum and isInsideCenter() function and
+ *                                       updated functions in 'mouseHandler' to prevent
+ *                                       errors when MB1 dragging tool off screen and to
+ *                                       only change cursor to hand in editable mode.
  * 
  * </pre>
  * 
@@ -127,60 +131,97 @@ public class AzimuthToolLayer extends
     private DrawableString[] labels = new DrawableString[RANGES.length
             + ANGLES.length];
 
+    /**
+     * Associated navigation modes:
+     * <UL>
+     * <LI>MOVE - Move the Az/Ran tool
+     * <LI>PAN - Allow other tools, such as pan, to have control
+     */
+    private static enum Mode {
+        MOVE, PAN
+    };
+
     private IInputHandler mouseHandler = new InputAdapter() {
 
         private Shell lastShell = null;
 
+        /** The mode of the mouse. By default, pan */
+        private Mode mode = Mode.PAN;
+
         @Override
         public boolean handleMouseUp(int x, int y, int mouseButton) {
-            if (nextCoordinate != null || (mouseButton == 3 && isEditable())) {
-                currCoordinate = getResourceContainer().translateClick(x, y);
-
-                if (currCoordinate == null) {
+            if (mouseButton == 3 && isEditable()) {
+                IDisplayPaneContainer container = getResourceContainer();
+                if (container == null) {
                     return false;
                 }
 
+                Coordinate c2 = container.translateClick(x, y);
+
+                if (c2 == null) {
+                    return false;
+                }
+
+                currCoordinate = c2;
                 centerPixel = descriptor.worldToPixel(new double[] {
                         currCoordinate.x, currCoordinate.y });
                 nextCoordinate = null;
 
-                if (lastShell != null) {
-                    lastShell.setCursor(lastShell.getDisplay().getSystemCursor(
-                            SWT.CURSOR_ARROW));
-                    lastShell = null;
-                }
-
                 recreate = true;
                 issueRefresh();
                 return true;
+            } else if (this.mode == Mode.PAN) {
+                return false;
+            } else if (this.mode == Mode.MOVE) {
+                recreate = true;
+                issueRefresh();
             }
-            return false;
+
+            // Default back to pan operation
+            mode = Mode.PAN;
+            return true;
         }
 
         @Override
         public boolean handleMouseDown(int x, int y, int mouseButton) {
-            if (isEditable() && mouseButton == 1 && lastShell != null) {
-                nextCoordinate = new Coordinate(x, y);
+
+            if (isEditable() && mouseButton == 1 && isInsideCenter(x, y)) {
+                this.mode = Mode.MOVE;
                 return true;
             }
+            issueRefresh();
             return false;
         }
 
         @Override
         public boolean handleMouseDownMove(int x, int y, int mouseButton) {
-            if (nextCoordinate != null) {
-                nextCoordinate = getResourceContainer().translateClick(x, y);
-                issueRefresh();
-                return true;
+            if (mouseButton != 1 || this.mode == Mode.PAN) {
+                return false;
             }
-            return false;
+
+            if (this.mode == Mode.MOVE) {
+                IDisplayPaneContainer container = getResourceContainer();
+                if (container == null) {
+                    return false;
+                }
+                Coordinate c2 = container.translateClick(x, y);
+
+                if (c2 == null) {
+                    return true;
+                }
+
+                currCoordinate = c2;
+                centerPixel = descriptor.worldToPixel(new double[] {
+                        currCoordinate.x, currCoordinate.y });
+
+                issueRefresh();
+            }
+            return true;
         }
 
         @Override
         public boolean handleMouseMove(int x, int y) {
-            if (StormTrackUIManager.getCoordinateIndex(getResourceContainer(),
-                    new Coordinate[] { currCoordinate }, new Coordinate(x, y),
-                    7.0) >= 0) {
+            if (isInsideCenter(x, y) && isEditable()) {
                 IWorkbenchWindow window = PlatformUI.getWorkbench()
                         .getActiveWorkbenchWindow();
                 lastShell = window.getShell();
@@ -193,7 +234,6 @@ public class AzimuthToolLayer extends
             }
             return false;
         }
-
     };
 
     public AzimuthToolLayer(GenericToolsResourceData<AzimuthToolLayer> data,
@@ -215,6 +255,25 @@ public class AzimuthToolLayer extends
         return AZIMUTH_LOCATION;
     }
 
+    /**
+     * Return if the given mouse position is within the tool's center.
+     * 
+     * @param refX
+     *            x location in screen pixels
+     * @param refY
+     *            y location in screen pixels
+     * @return boolean true if within center, false otherwise
+     */
+    public boolean isInsideCenter(int x, int y) {
+        IDisplayPaneContainer container = getResourceContainer();
+        if (container == null) {
+            return false;
+        }
+        int idx = ToolsUiUtil.closeToCoordinate(container,
+                new Coordinate[] { currCoordinate }, x, y, 7.0);
+        return idx >= 0;
+    }
+
     /*
      * (non-Javadoc)
      *