Changeset 5958 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManagerModel.java

Timestamp:

2013-05-13T03:10:05+02:00 (11 years ago)

Author:

Don-vip

Message:

• Improve PopupMenuLauncher to automatically select JTable/JList/JTree line under cursor
• Simplify all popup menus with this new feature
• Remove custom popup menus in BoundingBoxSelection and BoundingBoxSelection to use default text field popup menus recently introduced
• Fix EDT violations in changeset window
• Make Autoscale actions publicly available in MainMenu
• Add a new "Zoom to problem" mode in AutoScaleAction
• Make "Zoom to problem" menu item in validator dialog use this new feature
• Update enabled state of "Zoom to Conflict" and "Zoom to problem" against selection of conflict/validator dialog

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManagerModel.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManagerModel.java

@@ -19 +19 @@
 import org.openstreetmap.josm.data.osm.ChangesetCacheEvent;
 import org.openstreetmap.josm.data.osm.ChangesetCacheListener;
+import org.openstreetmap.josm.gui.util.GuiHelper;
 
 /**
@@ -104 +105 @@
      */
     public void setSelectedChangesets(Collection<Changeset> selected) {
-        selectionModel.clearSelection();
+        GuiHelper.runInEDTAndWait(new Runnable() {
+            @Override public void run() {
+                selectionModel.clearSelection();
+            }
+        });
         if (selected == null || selected.isEmpty())
             return;
         for (Changeset cs: selected) {
-            int idx = data.indexOf(cs);
+            final int idx = data.indexOf(cs);
             if (idx >= 0) {
-                selectionModel.addSelectionInterval(idx,idx);
+                GuiHelper.runInEDTAndWait(new Runnable() {
+                    @Override public void run() {
+                        selectionModel.addSelectionInterval(idx,idx);
+                    }
+                });
             }
         }
-    }
-
-    /**
-     * Selects the changeset displayed at row <code>row</code>
-     *
-     * @param row the row. Ignored if < 0 or >= {@link #getRowCount()}
-     */
-    public void setSelectedByIdx(int row) {
-        if (row < 0 || row >= getRowCount()) return;
-        selectionModel.setSelectionInterval(row, row);
     }