make marker.line.width dflt 1

... and use paper_bgcolor for marker.line.color dflt
plotly · Mar 4, 2019 · c1ccad6 · c1ccad6
1 parent a1d9b2b
commit c1ccad6
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 4 deletions.
diff --git a/src/traces/sunburst/attributes.js b/src/traces/sunburst/attributes.js
@@ -100,8 +100,14 @@ module.exports = {
         // },
 
         line: {
-            color: pieAtts.marker.line.color,
-            width: pieAtts.marker.line.width,
+            color: extendFlat({}, pieAtts.marker.line.color, {
+                dflt: null,
+                description: [
+                    'Sets the color of the line enclosing each sector.',
+                    'Defaults to the `paper_bgcolor` value.'
+                ].join(' ')
+            }),
+            width: extendFlat({}, pieAtts.marker.line.width, {dflt: 1}),
             editType: 'calc'
         },
         editType: 'calc'

diff --git a/src/traces/sunburst/defaults.js b/src/traces/sunburst/defaults.js
@@ -34,7 +34,7 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
     coerce('maxdepth');
 
     var lineWidth = coerce('marker.line.width');
-    if(lineWidth) coerce('marker.line.color');
+    if(lineWidth) coerce('marker.line.color', layout.paper_bgcolor);
 
     coerce('marker.colors');
 

diff --git a/test/image/baselines/sunburst_coffee.png b/test/image/baselines/sunburst_coffee.png
diff --git a/test/image/baselines/sunburst_first.png b/test/image/baselines/sunburst_first.png
diff --git a/test/image/baselines/sunburst_flare.png b/test/image/baselines/sunburst_flare.png
diff --git a/test/image/baselines/sunburst_values.png b/test/image/baselines/sunburst_values.png
diff --git a/test/image/mocks/sunburst_values.json b/test/image/mocks/sunburst_values.json
@@ -34,7 +34,8 @@
       "xanchor": "center",
       "y": 1.1,
       "yanchor": "bottom"
-    }]
+    }],
 
+    "paper_bgcolor": "#d3d3d3"
   }
 }