Changeset 8808 in josm for trunk/src/org

Timestamp:

2015-09-29T22:46:25+02:00 (9 years ago)

Author:

simon04

Message:

see #6963 - ParallelWayAction: use notification for invalid selection message

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java

@@ -32 +32 @@
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.gui.MapView;
+import org.openstreetmap.josm.gui.Notification;
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.gui.layer.MapViewPaintable;
@@ -554 +555 @@
             return true;
         } catch (IllegalArgumentException e) {
-            // TODO: Not ideal feedback. Maybe changing the cursor could be a good mechanism?
-            JOptionPane.showMessageDialog(
-                    Main.parent,
-                    tr("ParallelWayAction\n" +
-                            "The ways selected must form a simple branchless path"),
-                    tr("Make parallel way error"),
-                    JOptionPane.INFORMATION_MESSAGE);
+            new Notification(tr("ParallelWayAction\n" +
+                    "The ways selected must form a simple branchless path"))
+                    .setIcon(JOptionPane.INFORMATION_MESSAGE)
+                    .show();
             // The error dialog prevents us from getting the mouseReleased event
             resetMouseTrackingState();