Changeset 15769 in josm for trunk/src

Timestamp:

2020-01-26T15:43:41+01:00 (5 years ago)

Author:

simon04

Message:

see #17688 - TagInfoViewer: allow to select/copy multiple values

Location:

trunk/src/org/openstreetmap/josm/gui/history

Files:

• SelectionSynchronizer.java (modified) (1 diff)
• TagInfoViewer.java (modified) (2 diffs)

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

@@ -52 +52 @@
         DefaultListSelectionModel referenceModel = (DefaultListSelectionModel) e.getSource();
         int i = referenceModel.getMinSelectionIndex();
+        int j = referenceModel.getMaxSelectionIndex();
         for (ListSelectionModel model : participants) {
             if (model == e.getSource()) {
                 continue;
             }
-            model.setSelectionInterval(i, i);
+            model.setSelectionInterval(i, j);
         }
         preventRecursion = false;

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

@@ -71 +71 @@
         TagTableModel tagTableModel = model.getTagTableModel(pointInTime);
         JTable table = new JTable(tagTableModel, new TagTableColumnModel());
-        table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
+        table.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
         selectionSynchronizer.participateInSynchronizedSelection(table.getSelectionModel());
         table.setTransferHandler(new TagInfoTransferHandler());
@@ -88 +88 @@
 
         tagMenu.add(trackJosmAction(new CopyValueAction(table, tagKeyFn, objectSp)));
-        tagMenu.add(trackJosmAction(new CopyKeyValueAction(table, tagKeyFn, objectSp)));
+        final CopyKeyValueAction copyKeyValueAction = new CopyKeyValueAction(table, tagKeyFn, objectSp);
+        tagMenu.add(trackJosmAction(copyKeyValueAction));
+        tagMenu.addPopupMenuListener(copyKeyValueAction);
         tagMenu.add(trackJosmAction(new CopyAllKeyValueAction(table, tagKeyFn, objectSp)));
         tagMenu.addSeparator();