Skip to content

Commit 6cc9b16

Browse files
committed
bar: set marker.line.width dflt to 0
- so that plotting code - which relies on marker.line.width to set the effective 'bargap' does not error out - broken since #124
1 parent e246331 commit 6cc9b16

File tree

2 files changed

+17
-18
lines changed

2 files changed

+17
-18
lines changed

src/traces/bar/attributes.js

+16-8
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,19 @@ var extendFlat = require('../../lib/extend').extendFlat;
1515
var scatterMarkerAttrs = scatterAttrs.marker;
1616
var scatterMarkerLineAttrs = scatterMarkerAttrs.line;
1717

18+
var markerLineWidth = extendFlat({},
19+
scatterMarkerLineAttrs.width, { dflt: 0 });
20+
21+
var markerLine = extendFlat({}, {
22+
width: markerLineWidth
23+
}, colorAttributes('marker.line'));
24+
25+
var marker = extendFlat({}, {
26+
showscale: scatterMarkerAttrs.showscale,
27+
line: markerLine,
28+
}, colorAttributes('marker'));
29+
30+
1831
module.exports = {
1932
x: scatterAttrs.x,
2033
x0: scatterAttrs.x0,
@@ -23,6 +36,7 @@ module.exports = {
2336
y0: scatterAttrs.y0,
2437
dy: scatterAttrs.dy,
2538
text: scatterAttrs.text,
39+
2640
orientation: {
2741
valType: 'enumerated',
2842
role: 'info',
@@ -33,14 +47,8 @@ module.exports = {
3347
'along the vertical (horizontal).'
3448
].join(' ')
3549
},
36-
marker: extendFlat({}, {
37-
showscale: scatterMarkerAttrs.showscale,
38-
line: extendFlat({},
39-
{width: scatterMarkerLineAttrs.width},
40-
colorAttributes('marker.line')
41-
)},
42-
colorAttributes('marker')
43-
),
50+
51+
marker: marker,
4452

4553
r: scatterAttrs.r,
4654
t: scatterAttrs.t,

src/traces/histogram/attributes.js

+1-10
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ var colorAttributes = require('../../components/colorscale/color_attributes');
1313

1414
var extendFlat = require('../../lib/extend').extendDeep;
1515

16-
var barMarkerAttrs = barAttrs.marker;
17-
var barMarkerLineAttrs = barMarkerAttrs.line;
1816

1917
module.exports = {
2018
x: {
@@ -124,14 +122,7 @@ module.exports = {
124122
},
125123
ybins: makeBinsAttr('y'),
126124

127-
marker: extendFlat({}, {
128-
showscale: barMarkerAttrs.showscale,
129-
line: extendFlat({},
130-
{width: extendFlat({}, barMarkerLineAttrs.width, {dflt: 0})},
131-
colorAttributes('marker.line'))
132-
},
133-
colorAttributes('marker')
134-
),
125+
marker: barMarkerAttrs.marker,
135126

136127
_nestedModules: {
137128
'error_y': 'ErrorBars',

0 commit comments

Comments
 (0)