Changeset 4356 in josm for trunk

Timestamp:

2011-08-27T16:03:35+02:00 (13 years ago)

Author:

stoecker

Message:

convert some more ToggleDialog's

Location:

trunk/src/org/openstreetmap/josm/gui/dialogs

Files:

• LayerListDialog.java (modified) (6 diffs)
• properties/PropertiesDialog.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
-import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Component;
@@ -18 +17 @@
 import java.beans.PropertyChangeListener;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
@@ -31 +31 @@
 import javax.swing.JLabel;
 import javax.swing.JMenuItem;
-import javax.swing.JPanel;
 import javax.swing.JPopupMenu;
-import javax.swing.JScrollPane;
 import javax.swing.JSlider;
 import javax.swing.JTable;
@@ -114 +112 @@
 
     ActivateLayerAction activateLayerAction;
-
-    protected JPanel createButtonPanel() {
-        JPanel buttonPanel = getButtonPanel(5);
-
-        // -- move up action
-        MoveUpAction moveUpAction = new MoveUpAction();
-        adaptTo(moveUpAction, model);
-        adaptTo(moveUpAction,selectionModel);
-        buttonPanel.add(new SideButton(moveUpAction));
-
-        // -- move down action
-        MoveDownAction moveDownAction = new MoveDownAction();
-        adaptTo(moveDownAction, model);
-        adaptTo(moveDownAction,selectionModel);
-        buttonPanel.add(new SideButton(moveDownAction));
-
-        // -- activate action
-        activateLayerAction = new ActivateLayerAction();
-        adaptTo(activateLayerAction, selectionModel);
-        buttonPanel.add(new SideButton(activateLayerAction));
-
-        // -- show hide action
-        ShowHideLayerAction showHideLayerAction = new ShowHideLayerAction();
-        adaptTo(showHideLayerAction, selectionModel);
-        buttonPanel.add(new SideButton(showHideLayerAction));
-
-        //-- layer opacity action
-        LayerOpacityAction layerOpacityAction = new LayerOpacityAction();
-        adaptTo(layerOpacityAction, selectionModel);
-        opacityButton = new SideButton(layerOpacityAction);
-        buttonPanel.add(opacityButton);
-
-        // -- merge layer action
-        MergeAction mergeLayerAction = new MergeAction();
-        adaptTo(mergeLayerAction, model);
-        adaptTo(mergeLayerAction,selectionModel);
-        buttonPanel.add(new SideButton(mergeLayerAction));
-
-        // -- duplicate layer action
-        DuplicateAction duplicateLayerAction = new DuplicateAction();
-        adaptTo(duplicateLayerAction, model);
-        adaptTo(duplicateLayerAction, selectionModel);
-        buttonPanel.add(new SideButton(duplicateLayerAction));
-
-        //-- delete layer action
-        DeleteLayerAction deleteLayerAction = new DeleteLayerAction();
-        layerList.getActionMap().put("deleteLayer", deleteLayerAction);
-        adaptTo(deleteLayerAction, selectionModel);
-        buttonPanel.add(new SideButton(deleteLayerAction, false));
-
-        return buttonPanel;
-    }
 
     /** stores which layer index to toggle and executes the ShowHide action if the layer is present */
@@ -254 +200 @@
         }
 
-        add(new JScrollPane(layerList), BorderLayout.CENTER);
-
         // init the model
         //
@@ -275 +219 @@
                 );
 
-        add(createButtonPanel(), BorderLayout.SOUTH);
+        // -- move up action
+        MoveUpAction moveUpAction = new MoveUpAction();
+        adaptTo(moveUpAction, model);
+        adaptTo(moveUpAction,selectionModel);
+
+        // -- move down action
+        MoveDownAction moveDownAction = new MoveDownAction();
+        adaptTo(moveDownAction, model);
+        adaptTo(moveDownAction,selectionModel);
+
+        // -- activate action
+        activateLayerAction = new ActivateLayerAction();
+        adaptTo(activateLayerAction, selectionModel);
+
+        // -- show hide action
+        ShowHideLayerAction showHideLayerAction = new ShowHideLayerAction();
+        adaptTo(showHideLayerAction, selectionModel);
+
+        //-- layer opacity action
+        LayerOpacityAction layerOpacityAction = new LayerOpacityAction();
+        adaptTo(layerOpacityAction, selectionModel);
+        opacityButton = new SideButton(layerOpacityAction);
+
+        // -- merge layer action
+        MergeAction mergeLayerAction = new MergeAction();
+        adaptTo(mergeLayerAction, model);
+        adaptTo(mergeLayerAction,selectionModel);
+
+        // -- duplicate layer action
+        DuplicateAction duplicateLayerAction = new DuplicateAction();
+        adaptTo(duplicateLayerAction, model);
+        adaptTo(duplicateLayerAction, selectionModel);
+
+        //-- delete layer action
+        DeleteLayerAction deleteLayerAction = new DeleteLayerAction();
+        layerList.getActionMap().put("deleteLayer", deleteLayerAction);
+        adaptTo(deleteLayerAction, selectionModel);
+
+        createLayout(layerList, true, Arrays.asList(new SideButton[] {
+            new SideButton(moveUpAction),
+            new SideButton(moveDownAction),
+            new SideButton(activateLayerAction),
+            new SideButton(showHideLayerAction),
+            opacityButton,
+            new SideButton(mergeLayerAction),
+            new SideButton(duplicateLayerAction),
+            new SideButton(deleteLayerAction, false)
+        }));
+
         createVisibilityToggleShortcuts();
     }

trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

@@ -764 +764 @@
         JScrollPane scrollPane = new JScrollPane(bothTables);
         scrollPane.addMouseListener(dblClickWatch);
-        add(scrollPane, BorderLayout.CENTER);
 
         selectSth.setPreferredSize(scrollPane.getSize());
         presets.setSize(scrollPane.getSize());
-
-        JPanel buttonPanel = getButtonPanel(3);
 
         // -- add action and shortcut
         this.btnAdd = new SideButton(addAction);
         btnAdd.setFocusable(true);
-        buttonPanel.add(this.btnAdd);
         btnAdd.getInputMap(JComponent.WHEN_FOCUSED).put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), "onEnter");
         btnAdd.getActionMap().put("onEnter", addAction);
@@ -784 +780 @@
         membershipTable.getSelectionModel().addListSelectionListener(editAction);
         this.btnEdit = new SideButton(editAction);
-        buttonPanel.add(this.btnEdit);
 
         // -- delete action
@@ -796 +791 @@
         );
         getActionMap().put("delete", deleteAction);
-        buttonPanel.add(this.btnDel);
-        add(buttonPanel, BorderLayout.SOUTH);
+
+        createLayout(scrollPane, false, Arrays.asList(new SideButton[] {
+            this.btnAdd, this.btnEdit, this.btnDel
+        }));
 
         // -- help action