default fill to none if stroke

Author: mbostock

src/marks/area.js

@@ -23,8 +23,8 @@ export class Area extends Mark {
       ...options
     } = {}
   ) {
-    const [vfill, cfill] = maybeColor(fill, "currentColor");
     const [vstroke, cstroke] = maybeColor(stroke, "none");
+    const [vfill, cfill] = maybeColor(fill, cstroke === "none" ? "currentColor" : "none");
     if (z === undefined && vfill != null) z = vfill;
     if (z === undefined && vstroke != null) z = vstroke;
     super(

src/marks/bar.js

@@ -23,8 +23,8 @@ export class AbstractBar extends Mark {
       ...options
     } = {}
   ) {
-    const [vfill, cfill] = maybeColor(fill, "currentColor");
     const [vstroke, cstroke] = maybeColor(stroke, "none");
+    const [vfill, cfill] = maybeColor(fill, cstroke === "none" ? "currentColor" : "none");
     super(
       data,
       [

src/marks/rect.js

@@ -26,8 +26,8 @@ export class Rect extends Mark {
       ...options
     } = {}
   ) {
-    const [vfill, cfill] = maybeColor(fill, "currentColor");
     const [vstroke, cstroke] = maybeColor(stroke, "none");
+    const [vfill, cfill] = maybeColor(fill, cstroke === "none" ? "currentColor" : "none");
     super(
       data,
       [