Changeset 27168 in osm for applications/editors

Timestamp:

2011-11-28T11:47:56+01:00 (13 years ago)

Author:

larry0ua

Message:

'PicLayer - now markers are visible only when they are needed - active Layer and transform/add marker mapmode'

Location:

applications/editors/josm/plugins/piclayer

Files:

• build.xml (modified) (1 diff)
• src/org/openstreetmap/josm/plugins/piclayer/PicLayerPlugin.java (modified) (1 diff)
• src/org/openstreetmap/josm/plugins/piclayer/actions/GenericPicTransformAction.java (modified) (2 diffs)
• src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/MovePointAction.java (modified) (2 diffs)
• src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/TransformPointAction.java (modified) (1 diff)
• src/org/openstreetmap/josm/plugins/piclayer/layer/PicLayerAbstract.java (modified) (2 diffs)

applications/editors/josm/plugins/piclayer/build.xml

@@ -22 +22 @@
 -->
 <project name="PicLayer" default="dist" basedir=".">
-    <property name="commit.message" value="PicLayer - removed hiding buttons from PicLayer main menu - this is in core responsibility now"/>
+    <property name="commit.message" value="PicLayer - now markers are visible only when they are needed - active Layer and transform/add marker mapmode"/>
     <property name="plugin.main.version" value="4549"/>
     <!--

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/PicLayerPlugin.java

@@ -130 +130 @@
                 btn.setVisible(false);
             }
+            if (oldLayer != null)
+                ((PicLayerAbstract)oldLayer).setDrawPoints(false);
         }
         if (!oldPic && newPic) { // enter picture layer - reset visibility of controls

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/GenericPicTransformAction.java

@@ -54 +54 @@
             if ( Main.map.mapView.getActiveLayer() instanceof PicLayerAbstract ) {
                 currentLayer = (PicLayerAbstract)Main.map.mapView.getActiveLayer();
-        
+
                 if ( currentLayer != null && e.getButton() == MouseEvent.BUTTON1 ) {
                     isDragging = true;
@@ -88 +88 @@
                 return l instanceof PicLayerAbstract;
         }
-        
-        
+
+        protected void updateDrawPoints(boolean value) {
+            Layer active = Main.map.mapView.getActiveLayer();
+        if (active instanceof PicLayerAbstract) {
+            ((PicLayerAbstract)active).setDrawPoints(value);
+        }
+        Main.map.mapView.repaint();
+        }
+
 }

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/MovePointAction.java

@@ -32 +32 @@
                 }
         }
-        
+
         @Override
         public void mouseClicked(MouseEvent e) {
                 if (currentLayer == null)
                         return;
-                
+
                 try {
                         Point2D pressed = currentLayer.transformPoint(e.getPoint());
@@ -47 +47 @@
                 Main.map.mapView.repaint();
         }
+
+        @Override
+        public void enterMode() {
+            super.enterMode();
+            updateDrawPoints(true);
+        }
+
+        @Override
+        public void exitMode() {
+            super.exitMode();
+            updateDrawPoints(false);
+        }
 }

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/TransformPointAction.java

@@ -34 +34 @@
                 }
         }
+    @Override
+    public void enterMode() {
+        super.enterMode();
+        updateDrawPoints(true);
+    }
 
+    @Override
+    public void exitMode() {
+        super.exitMode();
+        updateDrawPoints(false);
+    }
 }

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/layer/PicLayerAbstract.java

@@ -77 +77 @@
     private Icon layerIcon = null;
 
+    private boolean drawPoints = true;
+
+    public void setDrawPoints(boolean value) {
+        drawPoints = value;
+    }
+
     private PictureTransform transformer;
 
@@ -239 +245 @@
                 );
             }
-            // draw points for selection
-            Graphics2D gPoints = (Graphics2D)g2.create();
-
-            gPoints.translate(pic_offset_x, pic_offset_y);
-
-            gPoints.setColor(Color.RED); // red color for points output
-
-            AffineTransform tr = AffineTransform.getScaleInstance(scalex, scaley);
-            tr.concatenate(transformer.getTransform());
-
-            for (Point2D p : transformer.getOriginPoints()) {
-               Point2D trP = tr.transform(p, null);
-               int x = (int)trP.getX(), y = (int)trP.getY();
-               gPoints.drawOval(x-2, y-2, 5, 5);
-               gPoints.drawImage(pinImage, x, y, null);
+            if (drawPoints ) {
+                // draw points for selection
+                Graphics2D gPoints = (Graphics2D)g2.create();
+
+                gPoints.translate(pic_offset_x, pic_offset_y);
+
+                gPoints.setColor(Color.RED); // red color for points output
+
+                AffineTransform tr = AffineTransform.getScaleInstance(scalex, scaley);
+                tr.concatenate(transformer.getTransform());
+
+                for (Point2D p : transformer.getOriginPoints()) {
+                   Point2D trP = tr.transform(p, null);
+                   int x = (int)trP.getX(), y = (int)trP.getY();
+                   gPoints.drawOval(x-2, y-2, 5, 5);
+                   gPoints.drawImage(pinImage, x, y, null);
+                }
             }
-
         } else {
             // TODO: proper logging