Changeset 16393 in josm for trunk/src/org

Timestamp:

2020-05-10T07:58:27+02:00 (5 years ago)

Author:

GerdP

Message:

fix #19181: Preference "Draw. Direction Arrows" looks ugly when zoomed out

• only force the drawing of the final arrow when the way is selected

File:

• trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

@@ -1325 +1325 @@
         boolean initialMoveToNeeded = true;
         ArrowPaintHelper drawArrowHelper = null;
+        double minSegmentLenSq = 0;
         if (showOrientation) {
             drawArrowHelper = new ArrowPaintHelper(PHI, 10 + line.getLineWidth());
+            minSegmentLenSq = Math.pow(drawArrowHelper.getOnLineLength() * 1.3, 2);
         }
         while (it.hasNext()) {
@@ -1342 +1344 @@
                 /* draw arrow */
                 if (drawArrowHelper != null) {
-                    boolean drawArrow;
-                    // always draw last arrow - no matter how short the segment is
-                    drawArrow = !it.hasNext();
-                    if (!showHeadArrowOnly) {
-                        // draw arrows in between only if there is enough space
-                        drawArrow = drawArrow || p1.distanceToInView(p2) > drawArrowHelper.getOnLineLength() * 1.3;
+                    final boolean drawArrow;
+                    if (way.isSelected()) {
+                        // always draw last arrow - no matter how short the segment is
+                        drawArrow = !it.hasNext() || p1.distanceToInViewSq(p2) > minSegmentLenSq;
+                    } else {
+                        // not selected: only draw arrow when it fits
+                        drawArrow = (!showHeadArrowOnly || !it.hasNext()) && p1.distanceToInViewSq(p2) > minSegmentLenSq;
                     }
                     if (drawArrow) {