Changeset 10258 in osm for applications/editors/josm

Timestamp:

2008-08-28T00:06:55+02:00 (16 years ago)

Author:

khris78

Message:

Agpifoj plugin : changed the buttons size in AgpifojDialog

Location:

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

Files:

• AgpifojDialog.java (modified) (7 diffs)
• ImageDisplay.java (modified) (2 diffs)

applications/editors/josm/plugins/agpifoj/src/org/openstreetmap/josm/plugins/agpifoj/AgpifojDialog.java

@@ -8 +8 @@
 
 import java.awt.BorderLayout;
+import java.awt.Dimension;
 import java.awt.FlowLayout;
 import java.awt.event.ActionEvent;
@@ -23 +24 @@
 
 public class AgpifojDialog extends ToggleDialog implements ActionListener {
+
+    private static final String COMMAND_ZOOM = "zoom";
+    private static final String COMMAND_CENTERVIEW = "centre";
+    private static final String COMMAND_NEXT = "next";
+    private static final String COMMAND_REMOVE = "remove";
+    private static final String COMMAND_PREVIOUS = "previous";
 
     private ImageDisplay imgDisplay = new ImageDisplay();
@@ -56 +63 @@
         JButton button;
         
+        Dimension buttonDim = new Dimension(26,26);
         button = new JButton();
         button.setIcon(ImageProvider.get("dialogs", "previous"));
-        button.setActionCommand("previous");
+        button.setActionCommand(COMMAND_PREVIOUS);
         button.setToolTipText(tr("Previous"));
         button.addActionListener(this);
+        button.setPreferredSize(buttonDim);
         buttons.add(button);
         
         button = new JButton();
         button.setIcon(ImageProvider.get("dialogs", "delete"));
-        button.setActionCommand("remove");
+        button.setActionCommand(COMMAND_REMOVE);
         button.setToolTipText(tr("Remove photo from layer"));
         button.addActionListener(this);
+        button.setPreferredSize(buttonDim);
         buttons.add(button);
         
         button = new JButton();
         button.setIcon(ImageProvider.get("dialogs", "next"));
-        button.setActionCommand("next");
+        button.setActionCommand(COMMAND_NEXT);
         button.setToolTipText(tr("Next"));
         button.addActionListener(this);
+        button.setPreferredSize(buttonDim);
         buttons.add(button);
         
         JToggleButton tb = new JToggleButton();
         tb.setIcon(ImageProvider.get("dialogs", "centreview"));
-        tb.setActionCommand("centre");
+        tb.setActionCommand(COMMAND_CENTERVIEW);
         tb.setToolTipText(tr("Center view"));
         tb.addActionListener(this);
+        tb.setPreferredSize(buttonDim);
         buttons.add(tb);
         
         button = new JButton();
         button.setIcon(ImageProvider.get("dialogs", "zoom-best-fit"));
-        button.setActionCommand("zoom");
+        button.setActionCommand(COMMAND_ZOOM);
         button.setToolTipText(tr("Zoom best fit and 1:1"));
         button.addActionListener(this);
+        button.setPreferredSize(buttonDim);
         buttons.add(button);
         
@@ -98 +111 @@
 
     public void actionPerformed(ActionEvent e) {
-        if ("next".equals(e.getActionCommand())) {
+        if (COMMAND_NEXT.equals(e.getActionCommand())) {
             if (currentLayer != null) {
                 currentLayer.showNextPhoto();
             }
-        } else if ("previous".equals(e.getActionCommand())) {
+        } else if (COMMAND_PREVIOUS.equals(e.getActionCommand())) {
             if (currentLayer != null) {
                 currentLayer.showPreviousPhoto();
             }
             
-        } else if ("centre".equals(e.getActionCommand())) {
+        } else if (COMMAND_CENTERVIEW.equals(e.getActionCommand())) {
             centerView = ((JToggleButton) e.getSource()).isSelected();
             if (centerView && currentEntry != null && currentEntry.pos != null) {
@@ -113 +126 @@
             }
             
-        } else if ("zoom".equals(e.getActionCommand())) {
+        } else if (COMMAND_ZOOM.equals(e.getActionCommand())) {
             imgDisplay.zoomBestFitOrOne();
             
-        } else if ("remove".equals(e.getActionCommand())) {
+        } else if (COMMAND_REMOVE.equals(e.getActionCommand())) {
             if (currentLayer != null) {
                currentLayer.removeCurrentPhoto();
@@ -125 +138 @@
 
     public static void showImage(AgpifojLayer layer, ImageEntry entry) {
-        if (INSTANCE == null) {
-            Main.main.map.addToggleDialog(new AgpifojDialog());
-        }
-        
-        if (INSTANCE != null) {
-            INSTANCE.displayImage(layer, entry);
-        }
-        
+        getInstance().displayImage(layer, entry);
     }
 
@@ -145 +151 @@
             }
         
-            if (centerView && entry != null && entry.pos != null) {
+            if (centerView && Main.map != null && entry != null && entry.pos != null) {
                 Main.map.mapView.zoomTo(entry.pos, Main.map.mapView.getScale());
             }

applications/editors/josm/plugins/agpifoj/src/org/openstreetmap/josm/plugins/agpifoj/ImageDisplay.java

@@ -480 +480 @@
                 int x = 3;
                 int y = 3;
+                String line;
                 while (pos > 0) {
-                    String line = osdText.substring(lastPos, pos);
+                    line = osdText.substring(lastPos, pos);
                     Rectangle2D lineSize = metrics.getStringBounds(line, g);
                     g.setColor(bkground);
@@ -491 +492 @@
                     pos = osdText.indexOf("\n", lastPos);
                 }
-                if (lastPos > 0) {
-                    String line = osdText.substring(lastPos);
-                    Rectangle2D lineSize = g.getFontMetrics(g.getFont()).getStringBounds(line, g);
-                    g.setColor(bkground);
-                    g.fillRect(x, y, (int) lineSize.getWidth(), (int) lineSize.getHeight());
-                    g.setColor(Color.black);
-                    g.drawString(line, x, y + ascent);
-                }
+
+                line = osdText.substring(lastPos);
+                Rectangle2D lineSize = g.getFontMetrics(g.getFont()).getStringBounds(line, g);
+                g.setColor(bkground);
+                g.fillRect(x, y, (int) lineSize.getWidth(), (int) lineSize.getHeight());
+                g.setColor(Color.black);
+                g.drawString(line, x, y + ascent);
             }
         }