Mapillary Layer: Fix issue with painting across tile borders

Signed-off-by: Taylor Smock <[email protected]>
mapillary · Feb 1, 2023 · c812fdb · c812fdb
1 parent 56b8ad8
commit c812fdb
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/main/java/org/openstreetmap/josm/plugins/mapillary/gui/layer/MapillaryLayer.java b/src/main/java/org/openstreetmap/josm/plugins/mapillary/gui/layer/MapillaryLayer.java
@@ -389,11 +389,13 @@ private void drawSequence(final Graphics2D g, final MapView mv, final IWay<?> se
         Point previousPoint = null;
         for (INode current : nodes) {
             final Point currentPoint = mv.getPoint(current);
-            g.setColor(forcedColor != null ? forcedColor : getColor(current));
+            if (MapillaryImageUtils.isImage(current)) {
+                g.setColor(forcedColor != null ? forcedColor : getColor(current));
+            }
             if (previous != null && (mv.contains(currentPoint) || mv.contains(previousPoint))) {
                 // FIXME get the right color for the line segment
+                final boolean visibleNode = zoom >= 14 && sequence.isVisible() && MapillaryImageUtils.isImage(previous);
                 g.drawLine(previousPoint.x, previousPoint.y, currentPoint.x, currentPoint.y);
-                final boolean visibleNode = zoom < 14 || sequence.isVisible() || MapillaryImageUtils.isImage(previous);
                 if (visibleNode) {
                     // FIXME draw the image here (avoid calculating the points twice)
                     drawImageMarker(originalTransform, selectedImage, g, previous, distPer100Pixel, false);
@@ -402,7 +404,7 @@ private void drawSequence(final Graphics2D g, final MapView mv, final IWay<?> se
             previous = current;
             previousPoint = currentPoint;
         }
-        final boolean visibleNode = zoom < 14 || sequence.isVisible() || MapillaryImageUtils.isImage(previous);
+        final boolean visibleNode = zoom >= 14 && sequence.isVisible() && MapillaryImageUtils.isImage(previous);
         if (visibleNode) {
             // FIXME draw the image here (avoid calculating the points twice)
             drawImageMarker(originalTransform, selectedImage, g, previous, distPer100Pixel, false);