Changeset 4598 in josm for trunk/src/org/openstreetmap/josm/gui/history/VersionTable.java

Timestamp:

2011-11-19T22:27:22+01:00 (12 years ago)

Author:

bastiK

Message:

history browser: changed selection of 2 compared version. WAS: double click and single click. NOW: 2 rows of radio buttons (inspired by Mediawiki)

File:

• trunk/src/org/openstreetmap/josm/gui/history/VersionTable.java (modified) (8 diffs)

trunk/src/org/openstreetmap/josm/gui/history/VersionTable.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
+import java.awt.Component;
 import java.awt.event.ActionEvent;
+import java.awt.event.ItemEvent;
+import java.awt.event.ItemListener;
 import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
@@ -10 +13 @@
 import java.util.Observer;
 
+import javax.swing.DefaultCellEditor;
 import javax.swing.DefaultListSelectionModel;
+import javax.swing.JCheckBox;
 import javax.swing.JPopupMenu;
+import javax.swing.JRadioButton;
 import javax.swing.JTable;
 import javax.swing.ListSelectionModel;
+import javax.swing.SwingConstants;
 import javax.swing.event.ListSelectionEvent;
 import javax.swing.event.ListSelectionListener;
 import javax.swing.event.TableModelEvent;
+import javax.swing.table.TableCellRenderer;
 
 import org.openstreetmap.josm.actions.AbstractInfoAction;
@@ -25 +33 @@
  * VersionTable shows a list of version in a {@see History} of an {@see OsmPrimitive}.
  *
- *
  */
 public class VersionTable extends JTable implements Observer{
@@ -31 +38 @@
 
     protected void build() {
-        setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
-        addMouseListener(new MouseHandler());
-        getSelectionModel().addListSelectionListener(new SelectionHandler());
+        setRowSelectionAllowed(false);
         popupMenu = new VersionTablePopupMenu();
         addMouseListener(new PopupMenuTrigger());
@@ -44 +49 @@
     }
 
-    protected void handleSelectReferencePointInTime(int row) {
-        getVersionTableModel().setReferencePointInTime(row);
-    }
-
-    protected void handleSelectCurrentPointInTime(int row) {
-        getVersionTableModel().setCurrentPointInTime(row);
-    }
-
     protected HistoryBrowserModel.VersionTableModel getVersionTableModel() {
         return (HistoryBrowserModel.VersionTableModel) getModel();
-    }
-
-    class MouseHandler extends MouseAdapter {
-        protected void handleDoubleClick(MouseEvent e) {
-            int row = rowAtPoint(e.getPoint());
-            handleSelectReferencePointInTime(row);
-        }
-
-        @Override
-        public void mouseClicked(MouseEvent e) {
-            switch(e.getClickCount()) {
-            case 2: handleDoubleClick(e); break;
-            }
-        }
-    }
-
-    class SelectionHandler implements ListSelectionListener {
-        public void valueChanged(ListSelectionEvent e) {
-            DefaultListSelectionModel model = (DefaultListSelectionModel)e.getSource();
-            // For some reason we receive multiple "adjusting" events here even when the source is a simple "set selection" action
-            // The last and proper event will have getValueIsAdjusting() == false
-            if (model.getMinSelectionIndex() >= 0 && e.getValueIsAdjusting() == false) {
-                handleSelectCurrentPointInTime(model.getMinSelectionIndex());
-            }
-        }
     }
 
@@ -85 +57 @@
     }
 
-    /* (non-Javadoc)
-     * @see javax.swing.JTable#tableChanged(javax.swing.event.TableModelEvent)
-     */
-    @Override
-    public void tableChanged(TableModelEvent e) {
-        int selectedRow = getSelectedRow();
-        super.tableChanged(e);
-        // Restore list selection (fix #6899)
-        if (selectedRow > -1 && selectedRow < getRowCount()) {
-            getSelectionModel().setSelectionInterval(0, selectedRow);
-        }
-    }
-
     protected void showPopupMenu(MouseEvent evt) {
         HistoryBrowserModel.VersionTableModel model = getVersionTableModel();
-        int row = getSelectedRow();
-        if (row == -1) {
-            row = rowAtPoint(evt.getPoint());
-        }
+        int row = rowAtPoint(evt.getPoint());
         if (!model.isLatest(row)) {
             HistoryOsmPrimitive primitive = model.getPrimitive(row);
@@ -109 +65 @@
             popupMenu.show(evt.getComponent(), evt.getX(), evt.getY());
         }
-    }
-
-    public void updateSelection() {
-        final int row = getVersionTableModel().getRowCount() - 1;
-        setRowSelectionInterval(row, row);
     }
 
@@ -180 +131 @@
         }
     }
+
+    public static class RadioButtonRenderer extends JRadioButton implements TableCellRenderer {
+
+        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus,int row,int column) {
+            HistoryBrowserModel.VersionTableModel model = (HistoryBrowserModel.VersionTableModel)table.getModel();
+            setSelected(value != null && (Boolean)value);
+            setHorizontalAlignment(SwingConstants.CENTER);
+            return this;
+        }
+    }
+
+    public static class RadioButtonEditor extends DefaultCellEditor implements ItemListener {
+
+        private JRadioButton btn;
+
+        public RadioButtonEditor() {
+            super(new JCheckBox());
+            btn = new JRadioButton();
+            btn.setHorizontalAlignment(SwingConstants.CENTER);
+        }
+
+        public Component getTableCellEditorComponent(JTable table, Object value, boolean isSelected, int row, int column) {
+            if (value == null) return null;
+            boolean val = (Boolean) value;
+            btn.setSelected(val);
+            btn.addItemListener(this);
+            return btn;
+        }
+
+        public Object getCellEditorValue() {
+            btn.removeItemListener(this);
+            return btn.isSelected();
+        }
+
+        public void itemStateChanged(ItemEvent e) {
+            fireEditingStopped();
+        }
+    }
 }