Changeset 27166 in osm for applications/editors/josm/plugins/piclayer/src/org

Timestamp:

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

Author:

larry0ua

Message:

'PicLayer - removed hiding buttons from PicLayer main menu - this is in core responsibility now'

Location:

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer

Files:

• ActionVisibilityChangeMenu.java (modified) (2 diffs)
• PicLayerPlugin.java (modified) (4 diffs)
• PicToggleButton.java (modified) (1 diff)

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

@@ -5 +5 @@
 import java.awt.event.ActionEvent;
 
-import javax.swing.AbstractAction;
 import javax.swing.JCheckBoxMenuItem;
 import javax.swing.JMenu;
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.actions.JosmAction;
 
 enum PicActions {MOVE_PICTURE, MOVE_POINT, TRANSFORM_POINT, SCALEX, SCALEY, SCALEXY, SHEAR, ROTATE}
@@ -29 +29 @@
         super();
         setSelected(Main.pref.getBoolean(button.getVisibilityKey(), button.getDefVisibility()));
-        setAction(new AbstractAction() {
-            @Override
-            public void actionPerformed(ActionEvent e) {
-                boolean val = !Main.pref.getBoolean(button.getVisibilityKey(), button.getDefVisibility());
-                Main.pref.put(button.getVisibilityKey(), val);
-                SwitchVisibilityMenuItem.this.setSelected(val);
-                button.setVisible(val);
-            }
-        });
-        setText(tr(button.getBtnName()));
+        setAction(new ButtonAction(button));
+    }
+    class ButtonAction extends JosmAction {
+        private PicToggleButton button;
+
+        public ButtonAction(PicToggleButton button) {
+            super(button.getBtnName(), null, button.getBtnName(), null, false, false);
+            this.button = button;
+        }
+
+        @Override
+        public void actionPerformed(ActionEvent e) {
+            boolean val = !Main.pref.getBoolean(button.getVisibilityKey(), button.getDefVisibility());
+            Main.pref.put(button.getVisibilityKey(), val);
+            SwitchVisibilityMenuItem.this.setSelected(val);
+            button.setVisible(val);
+        }
+
     }
 }

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

@@ -23 +23 @@
 import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
 import static org.openstreetmap.josm.tools.I18n.marktr;
+import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.awt.event.KeyEvent;
@@ -59 +60 @@
     // Plugin menu
     private JMenu menu = null;
-    private ActionVisibilityChangeMenu actionVisibility;
 
     /**
@@ -102 +102 @@
 
             buttonList = new ArrayList<PicToggleButton>(7);
-            buttonList.add(new PicToggleButton(movePictureAction, "Move Picture", "piclayer.actionvisibility.move", true));
-            buttonList.add(new PicToggleButton(movePointAction, "Move Point", "piclayer.actionvisibility.movepoint", true));
-            buttonList.add(new PicToggleButton(transformPointAction, "Transform Point", "piclayer.actionvisibility.transformpoint", true));
-            buttonList.add(new PicToggleButton(rotatePictureAction, "Rotate", "piclayer.actionvisibility.rotate", false));
-            buttonList.add(new PicToggleButton(scaleXYPictureAction, "Scale", "piclayer.actionvisibility.scale", false));
-            buttonList.add(new PicToggleButton(scaleXPictureAction, "Scale X", "piclayer.actionvisibility.scalex", false));
-            buttonList.add(new PicToggleButton(scaleYPictureAction, "Scale Y", "piclayer.actionvisibility.scaley", false));
-            buttonList.add(new PicToggleButton(shearPictureAction, "Shear", "piclayer.actionvisibility.shear", false));
+            buttonList.add(new PicToggleButton(movePictureAction, tr("Move Picture"), "piclayer.actionvisibility.move", true));
+            buttonList.add(new PicToggleButton(movePointAction, tr("Move Point"), "piclayer.actionvisibility.movepoint", true));
+            buttonList.add(new PicToggleButton(transformPointAction, tr("Transform Point"), "piclayer.actionvisibility.transformpoint", true));
+            buttonList.add(new PicToggleButton(rotatePictureAction, tr("Rotate"), "piclayer.actionvisibility.rotate", false));
+            buttonList.add(new PicToggleButton(scaleXYPictureAction, tr("Scale"), "piclayer.actionvisibility.scale", false));
+            buttonList.add(new PicToggleButton(scaleXPictureAction, tr("Scale X"), "piclayer.actionvisibility.scalex", false));
+            buttonList.add(new PicToggleButton(scaleYPictureAction, tr("Scale Y"), "piclayer.actionvisibility.scaley", false));
+            buttonList.add(new PicToggleButton(shearPictureAction, tr("Shear"), "piclayer.actionvisibility.shear", false));
 
             for(IconToggleButton btn : buttonList) {
                 newFrame.addMapMode(btn);
             }
-
-            if (actionVisibility == null)
-                menu.add(actionVisibility = new ActionVisibilityChangeMenu());
         }
     }
@@ -129 +126 @@
         // actually that should be not enough - JOSM should hide all buttons that are disabled for current layer!
         if (oldPic && !newPic || oldLayer == null && !newPic) { // leave picture layer - hide all controls
-            for (PicToggleButton btn : buttonList)
+            for (PicToggleButton btn : buttonList) {
+                btn.writeVisible();
                 btn.setVisible(false);
+            }
         }
         if (!oldPic && newPic) { // enter picture layer - reset visibility of controls

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

@@ -34 +34 @@
         setVisible(Main.pref.getBoolean(getVisibilityKey(), getDefVisibility()));
     }
+    public void writeVisible() {
+        Main.pref.put(getVisibilityKey(), isVisible());
+    }
 
 }