Changeset 35916 in osm for applications

Timestamp:

2022-02-09T19:35:36+01:00 (3 years ago)

Author:

taylor.smock

Message:

buildings_tools: fix #14213: "Rotate crosshair" forgets rotation after right-click drag

patch by Woazboat

File:

• applications/editors/josm/plugins/buildings_tools/src/org/openstreetmap/josm/plugins/buildings_tools/DrawBuildingAction.java (modified) (3 diffs)

applications/editors/josm/plugins/buildings_tools/src/org/openstreetmap/josm/plugins/buildings_tools/DrawBuildingAction.java

@@ -55 +55 @@
     private final Cursor cursorJoinNode;
     private final Cursor cursorJoinWay;
-    private Cursor currCursor;
     private Cursor customCursor;
 
@@ -81 +80 @@
                 ImageProvider.getCursor("crosshair", null));
 
-        currCursor = getCursor();
         cursorJoinNode = ImageProvider.getCursor("crosshair", "joinnode");
         cursorJoinWay = ImageProvider.getCursor("crosshair", "joinway");
@@ -105 +103 @@
      */
     private void setCursor(final Cursor c) {
-        if (currCursor.equals(c))
-            return;
-        try {
-            // We invoke this to prevent strange things from happening
-            EventQueue.invokeLater(() -> {
-                MapFrame map = MainApplication.getMap();
-                // Don't change cursor when mode has changed already
-                if (map == null || !(map.mapMode instanceof DrawBuildingAction))
-                    return;
-                map.mapView.setCursor(c);
-            });
-            currCursor = c;
-        } catch (Exception e) {
-            Logging.error(e);
-        }
+        MainApplication.getMap().mapView.setNewCursor(c, this);
     }