Changeset 19641 in josm

Timestamp:

2026-10-06T11:40:41+02:00 (96 minutes ago)

Author:

gaben

Message:

fix #18131 - antialias clipped SVG images (e.g. the JOSM logo)

Location:

trunk

Files:

• src/org/openstreetmap/josm/tools/ImageProvider.java (modified) (2 diffs)
• test/functional/org/openstreetmap/josm/tools/ImageProviderTest.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

@@ -83 +83 @@
 import com.github.weisj.jsvg.parser.LoaderContext;
 import com.github.weisj.jsvg.parser.SVGLoader;
+import com.github.weisj.jsvg.renderer.SVGRenderingHints;
 
 /**
@@ -1478 +1479 @@
         }
         return resizeMode.createBufferedImage(dim, new Dimension((int) sourceWidth, (int) sourceHeight), g -> {
+            // jsvg clips through Graphics2D#clip by default, which is not antialiased (jagged edges, see #18131).
+            // Soft clipping avoids that, but needs accurate mask/clip rendering, otherwise the clip is applied as a
+            // paint that the clipped elements' own paints override and parts of the image go missing.
+            g.setRenderingHint(SVGRenderingHints.KEY_SOFT_CLIPPING, SVGRenderingHints.VALUE_SOFT_CLIPPING_ON);
+            g.setRenderingHint(SVGRenderingHints.KEY_MASK_CLIP_RENDERING, SVGRenderingHints.VALUE_MASK_CLIP_RENDERING_ACCURACY);
             svg.render(null, g);
         }, null);

trunk/test/functional/org/openstreetmap/josm/tools/ImageProviderTest.java

@@ -6 +6 @@
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.openstreetmap.josm.testutils.ImageTestUtils.assertImageEquals;
 
@@ -129 +130 @@
         node.put("amenity", "fuel");
         assertDoesNotThrow(() -> OsmPrimitiveImageProvider.getResource(node, Collections.emptyList()));
+    }
+
+    /**
+     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/18131">#18131</a>
+     * <p>
+     * The JOSM logo is assembled from clipped paths. Unless the SVG renderer clips with antialiasing,
+     * the outline of the logo shows jagged steps.
+     */
+    @Test
+    void testTicket18131() {
+        ImageIcon icon = new ImageProvider("logo").setSize(new Dimension(256, 256)).get();
+        assertNotNull(icon);
+        BufferedImage image = (BufferedImage) icon.getImage();
+        // For every column, look at the topmost visible pixel: a fully opaque one means a hard, aliased edge
+        int opaqueEdges = 0;
+        int blendedEdges = 0;
+        for (int x = 0; x < image.getWidth(); x++) {
+            for (int y = 0; y < image.getHeight(); y++) {
+                int alpha = image.getRGB(x, y) >>> 24;
+                if (alpha == 0) {
+                    continue;
+                }
+                if (alpha == 0xff) {
+                    opaqueEdges++;
+                } else {
+                    blendedEdges++;
+                }
+                break;
+            }
+        }
+        assertTrue(opaqueEdges * 4 < blendedEdges,
+                "expected an antialiased logo outline, but found " + opaqueEdges + " aliased and "
+                        + blendedEdges + " antialiased edge pixels");
     }