Changeset 9253 in josm for trunk/src/org/openstreetmap/josm

Timestamp:

2016-01-02T02:06:16+01:00 (8 years ago)

Author:

stoecker

Message:

see #10684 - use squared toolbar buttons always, move side button image size definition to ImageProvider

Location:

trunk/src/org/openstreetmap/josm

Files:

• gui/SideButton.java (modified) (1 diff)
• gui/preferences/ToolbarPreferences.java (modified) (1 diff)
• tools/ImageProvider.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/SideButton.java

@@ -25 +25 @@
  */
 public class SideButton extends JButton implements Destroyable {
-    private static final int iconHeight = 20;
+    private static final int iconHeight = ImageProvider.getImageSizes(ImageProvider.ImageSizes.SIDEBUTTON).height;
 
     private transient PropertyChangeListener propertyChangeListener;

trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

@@ -1060 +1060 @@
                 if (i != null) {
                     b.setIcon(i);
+                    Dimension s = b.getPreferredSize();
+                    /* make squared toolbar icons */
+                    if (s.width < s.height) {
+                        s.width = s.height;
+                        b.setMinimumSize(s);
+                        b.setMaximumSize(s);
+                        //b.setSize(s);
+                    } else if (s.height < s.width) {
+                        s.height = s.width;
+                        b.setMinimumSize(s);
+                        b.setMaximumSize(s);
+                    }
                 } else {
                     // hide action text if an icon is set later (necessary for delayed/background image loading)

trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

@@ -150 +150 @@
          * @since 8323
          */
-        LAYER
+        LAYER,
+        /** Toolbar button icon size
+         * @since 9253
+         */
+        TOOLBAR,
+        /** Side button maximum height
+         * @since 9253
+         */
+        SIDEBUTTON
     }
 
@@ -348 +356 @@
         case MAPMAX: sizeval = Main.pref.getInteger("iconsize.mapmax", 48); break;
         case MAP: sizeval = Main.pref.getInteger("iconsize.mapmax", 16); break;
+        case SIDEBUTTON: sizeval = Main.pref.getInteger("iconsize.sidebutton", 20); break;
+        case TOOLBAR: /* TOOLBAR is LARGELICON - only provided in case of future changes */
         case POPUPMENU: /* POPUPMENU is LARGELICON - only provided in case of future changes */
         case LARGEICON: sizeval = Main.pref.getInteger("iconsize.largeicon", 24); break;