Changeset 4556 in josm

Timestamp:

2011-11-02T09:47:07+01:00 (13 years ago)

Author:

stoecker

Message:

fix #6971 - patch by olejorgenb - add tooltips to checkmarks in author dialog

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/UserListDialog.java (modified) (5 diffs)

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

@@ -5 +5 @@
 import static org.openstreetmap.josm.tools.I18n.trn;
 
-import java.awt.BorderLayout;
 import java.awt.Component;
 import java.awt.event.ActionEvent;
@@ -27 +26 @@
 
 import javax.swing.AbstractAction;
-import javax.swing.ImageIcon;
 import javax.swing.JLabel;
 import javax.swing.JOptionPane;
@@ -96 +94 @@
             @Override
             public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
-                final JLabel label = (JLabel) super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
-                label.setIcon((ImageIcon)value);
-                label.setText("");
-                return label;
-            };
+                // see http://download.oracle.com/javase/6/docs/api/javax/swing/table/DefaultTableCellRenderer.html#override
+                // for why we don't use the label directly
+                final JLabel renderLabel = (JLabel) super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
+                JLabel sourceLabel = (JLabel) value;
+                renderLabel.setIcon(sourceLabel.getIcon());
+                renderLabel.setText("");
+                renderLabel.setToolTipText(sourceLabel.getToolTipText());
+                return renderLabel;
+            }
         });
 
@@ -328 +330 @@
     static class UserTableModel extends DefaultTableModel {
         private ArrayList<UserInfo> data;
-        private ImageIcon greenCheckmark;
-        private ImageIcon greyCheckmark;
-        private ImageIcon redX;
+        private JLabel greenCheckmark;
+        private JLabel greyCheckmark;
+        private JLabel redX;
+        private JLabel empty;
 
         public UserTableModel() {
             setColumnIdentifiers(new String[]{tr("Author"),tr("# Objects"),"%", tr("CT")});
             data = new ArrayList<UserInfo>();
-            greenCheckmark = ImageProvider.get("misc", "green_check.png");
-            greyCheckmark = ImageProvider.get("misc", "grey_check.png");
-            redX = ImageProvider.get("misc", "red_x.png");
+            greenCheckmark = new JLabel(ImageProvider.get("misc", "green_check.png"));
+            greenCheckmark.setToolTipText(tr("Accepted"));
+            greyCheckmark = new JLabel(ImageProvider.get("misc", "grey_check.png"));
+            greyCheckmark.setToolTipText("Auto-accepted");
+            redX = new JLabel(ImageProvider.get("misc", "red_x.png"));
+            redX.setToolTipText("Declined");
+            empty = new JLabel("");
+            empty.setToolTipText("Undecided");
         }
 
@@ -383 +391 @@
                 case User.STATUS_AUTO_AGREED: return greyCheckmark;
                 case User.STATUS_NOT_AGREED: return redX;
-                default: return null;
+                default: return empty; // Undecided or unknown?
                 }
             }