Changeset 32078 in osm for applications/editors/josm/plugins/mapillary/src

Timestamp:

2016-02-19T23:36:27+01:00 (9 years ago)

Author:

floscher

Message:

[mapillary] Highlight the sequence that the selected image is contained in

• this also fixes one unit test that failed since the last commit
• and the JOSM-preferences are no longer overwritten everytime you run ./gradlew runJosm, only when you ./gradlew clean

Location:

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary

Files:

• MapillaryAbstractImage.java (modified) (1 diff)
• MapillaryLayer.java (modified) (1 diff)

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryAbstractImage.java

@@ -17 +17 @@
  *
  */
-public class MapillaryAbstractImage implements Comparable<MapillaryAbstractImage>{
+public class MapillaryAbstractImage implements Comparable<MapillaryAbstractImage> {
   /**
    * If two values for field ca differ by less than EPSILON both values are considered equal.

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryLayer.java

@@ -318 +318 @@
     }
     // Draw sequence line
-    g.setColor(Color.WHITE);
-    g.setStroke(new BasicStroke(this == Main.map.mapView.getActiveLayer() ? 3 : 1));
+    final MapillaryAbstractImage selectedImage = MapillaryLayer.getInstance().getData().getSelectedImage();
+    String selectedImageKey = null; // Intentionally not null to avoid null-check before .equals()
+    if (selectedImage != null && selectedImage.getSequence() != null) {
+      selectedImageKey = selectedImage.getSequence().getKey();
+    }
+    g.setStroke(new BasicStroke(this == Main.map.mapView.getActiveLayer() ? 3 : 2));
     for (MapillarySequence seq : getData().getSequences()) {
+      if (selectedImageKey != null && selectedImageKey.equals(seq.getKey())) {
+        g.setColor(Color.MAGENTA.brighter());
+      } else {
+        g.setColor(Color.WHITE);
+      }
       g.draw(MapViewGeometryUtil.getSequencePath(mv, seq));
     }