Changeset 9061 in josm for trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

Timestamp:

2015-11-24T20:49:07+01:00 (8 years ago)

Author:

bastiK

Message:

mappaint partial fill: render unclosed areas differently (margin on both sites) (see #12104)

File:

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

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

@@ -333 +333 @@
     private boolean showIcons;
     private boolean isOutlineOnly;
+    private boolean isUnclosedAreaHighlight;
+    private double unclosedAreaHighlightWidth;
 
     private Font orderFont;
@@ -451 +453 @@
     }
 
-    protected void drawArea(OsmPrimitive osm, Path2D.Double path, Color color, MapImage fillImage, Float extent,
+    /**
+     * Worker function for drawing areas.
+     * 
+     * @param osm the primitive
+     * @param path the path object for the area that should be drawn; in case
+     * of multipolygons, this can path can be a complex shape with one outer
+     * polygon and one or more inner polygons
+     * @param color The color to fill the area with.
+     * @param fillImage The image to fill the area with. Overrides color.
+     * @param extent if not null, area will be filled partially; specifies, how
+     * far to fill from the boundary towards the center of the area;
+     * if null, area will be filled completely
+     * @param unClosedHighlight true, if the fact that the way / multipolygon is not
+     * properly closed should be highlighted; this parameter is only used
+     * for partial fill ({@code extent != null}), otherwise it is ignored 
+     * @param disabled If this should be drawn with a special disabled style.
+     * @param text The text to write on the area.
+     */
+    protected void drawArea(OsmPrimitive osm, Path2D.Double path, Color color, MapImage fillImage, Float extent, boolean unClosedHighlight,
             boolean disabled, TextElement text) {
 
@@ -466 +486 @@
                     g.fill(area);
                 } else {
-                    Shape clip = g.getClip();
-                    g.clip(area);
-                    g.setStroke(new BasicStroke(2 * extent, BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER));
-                    g.draw(area);
-                    g.setClip(clip);
+                    if (unClosedHighlight) {
+                        g.setStroke(new BasicStroke((int)(unclosedAreaHighlightWidth / 100 * extent),
+                                BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER));
+                        g.draw(area);
+                    } else {
+                        Shape clip = g.getClip();
+                        g.clip(area);
+                        g.setStroke(new BasicStroke(2 * extent, BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER));
+                        g.draw(area);
+                        g.setClip(clip);
+                    }
                 }
             } else {
@@ -483 +509 @@
                     g.fill(area);
                 } else {
-                    Shape clip = g.getClip();
-                    BasicStroke stroke = new BasicStroke(2 * extent, BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER);
-                    g.clip(stroke.createStrokedShape(area));
-                    g.fill(area);
-                    g.setClip(clip);
+                    if (unClosedHighlight) {
+                        g.setStroke(new BasicStroke((int)(unclosedAreaHighlightWidth / 100 * extent),
+                                BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER));
+                        g.draw(area);
+                    } else {
+                        Shape clip = g.getClip();
+                        BasicStroke stroke = new BasicStroke(2 * extent, BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER);
+                        g.clip(stroke.createStrokedShape(area));
+                        g.fill(area);
+                        g.setClip(clip);
+                    }
                 }
                 g.setPaintMode();
@@ -592 +624 @@
                 drawArea(r, p,
                         pd.selected ? paintSettings.getRelationSelectedColor(color.getAlpha()) : color,
-                        fillImage, extent, disabled, text);
+                        fillImage, extent,
+                        isUnclosedAreaHighlight && extent != null && !pd.isClosed(),
+                        disabled, text);
             }
         }
@@ -609 +643 @@
      */
     public void drawArea(Way w, Color color, MapImage fillImage, Float extent, boolean disabled, TextElement text) {
-        drawArea(w, getPath(w), color, fillImage, extent, disabled, text);
+        drawArea(w, getPath(w), color, fillImage, extent, isUnclosedAreaHighlight && !w.isClosed(), disabled, text);
     }
 
@@ -1464 +1498 @@
         showIcons = paintSettings.getShowIconsDistance() > circum;
         isOutlineOnly = paintSettings.isOutlineOnly();
+        isUnclosedAreaHighlight = paintSettings.isUnclosedAreaHighlight();
+        unclosedAreaHighlightWidth = paintSettings.getUnclosedAreaHighlightWidth();
         orderFont = new Font(Main.pref.get("mappaint.font", "Droid Sans"), Font.PLAIN, Main.pref.getInteger("mappaint.fontsize", 8));