Changeset 5006 in josm for trunk

Timestamp:

2012-02-20T21:05:09+01:00 (13 years ago)

Author:

stoecker

Message:

make shortcut table a bit more colorful

Location:

trunk/src/org/openstreetmap/josm/gui/preferences/shortcut

Files:

• PrefJPanel.java (modified) (6 diffs)
• ShortcutPreference.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/PrefJPanel.java

@@ -2 +2 @@
 package org.openstreetmap.josm.gui.preferences.shortcut;
 
+import java.awt.Color;
+import java.awt.Component;
 import java.awt.Dimension;
 import java.awt.GridBagConstraints;
@@ -8 +10 @@
 import java.awt.Toolkit;
 
+import static org.openstreetmap.josm.tools.I18n.marktr;
 import static org.openstreetmap.josm.tools.I18n.tr;
 
@@ -40 +43 @@
 import javax.swing.table.AbstractTableModel;
 import javax.swing.table.TableModel;
+import javax.swing.table.DefaultTableCellRenderer;
+import javax.swing.table.TableColumnModel;
 
 import javax.swing.table.TableRowSorter;
@@ -144 +149 @@
     }
 
+    private static class ShortcutTableCellRenderer extends DefaultTableCellRenderer {
+
+        private boolean name;
+
+        public ShortcutTableCellRenderer(boolean name) {
+            this.name = name;
+        }
+
+        @Override
+        public Component getTableCellRendererComponent(JTable table, Object value, boolean
+                isSelected, boolean hasFocus, int row, int column) {
+            if(value instanceof Shortcut)
+            {
+                Shortcut sc = (Shortcut) value;
+                JLabel label = (JLabel) super.getTableCellRendererComponent(
+                    table, name ? sc.getLongText() : sc.getKeyText(), isSelected, hasFocus, row, column);
+                label.setBackground(Main.pref.getUIColor("Table.background"));
+                if (isSelected) {
+                    label.setForeground(Main.pref.getUIColor("Table.foreground"));
+                }
+                if(sc.getAssignedUser()) {
+                    label.setBackground(Main.pref.getColor(
+                            marktr("Shortcut Background: User"),
+                            new Color(200,255,200)));
+                } else if(!sc.getAssignedDefault()) {
+                    label.setBackground(Main.pref.getColor(
+                            marktr("Shortcut Background: Modified"),
+                            new Color(255,255,200)));
+                }
+                return label;
+            }
+            return null;
+        }
+    }
+
     private void initComponents() {
         JPanel listPane = new JPanel();
@@ -160 +200 @@
         shortcutTable.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
         shortcutTable.setAutoCreateRowSorter(true);
+        TableColumnModel mod = shortcutTable.getColumnModel();
+        mod.getColumn(0).setCellRenderer(new ShortcutTableCellRenderer(true));
+        mod.getColumn(1).setCellRenderer(new ShortcutTableCellRenderer(false));
         listScrollPane.setViewportView(shortcutTable);
 
@@ -268 +311 @@
                     actionPerformed(null);
                 }
-                model.fireTableCellUpdated(row, 1);
+                model.fireTableRowsUpdated(row, row);
             } else {
                 panel.disableAllModifierCheckboxes();

trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/ShortcutPreference.java

@@ -65 +65 @@
         }
         public Object getValueAt(int row, int col) {
-            Shortcut sc = data.get(row);
-            if (col == 0)
-                return sc.getLongText();
-            else if (col == 1)
-                return sc.getKeyText();
-            else
-                // This is a kind of hack that allows the actions on the editing controls
-                // to access the underlying shortcut object without introducing another
-                // method. I opted to stay within the interface.
-                return sc;
+            return data.get(row);
         }
         @Override