id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
10014	[Patch] Add support for canvas{fill-color}, deprecate canvas{background-color}	simon04	team	"According to https://wiki.osm.org/wiki/MapCSS/0.2#Canvas_properties, the ""official"" tag for setting the background color using MapCSS is
{{{
#!mapcss
canvas{
  fill-color: #abcdef;
}
}}}
instead of
{{{
#!mapcss
canvas{
  background-color: #abcdef;
}
}}}

Therefore, I suggest to support the standard and deprecate the current implementation.

{{{
#!patch
diff --git a/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java b/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java
index 0855a69..1e30485 100644
--- a/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java
+++ b/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java
@@ -202,7 +202,13 @@ public class MapCSSStyleSource extends StyleSource {
 
     private void loadCanvas() {
         Cascade c = constructSpecial(""canvas"");
-        backgroundColorOverride = c.get(""background-color"", null, Color.class);
+        backgroundColorOverride = c.get(""fill-color"", null, Color.class);
+        if (backgroundColorOverride == null) {
+            backgroundColorOverride = c.get(""background-color"", null, Color.class);
+            if (backgroundColorOverride != null) {
+                Main.warn(""Detected deprecated canvas{background-color} which will be removed shortly."");
+            }
+        }
     }
 
     private Cascade constructSpecial(String type) {
}}}"	defect	closed	normal	14.05	Core mappaint		fixed	mapcss	bastiK skunk@…