Skip to content

Commit c6b8d34

Browse files
committed
etienne default fixes
1 parent 29c369b commit c6b8d34

File tree

1 file changed

+5
-17
lines changed

1 file changed

+5
-17
lines changed

src/traces/parcoords/defaults.js

Lines changed: 5 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ var hasColorscale = require('../../components/colorscale/has_colorscale');
1414
var colorscaleDefaults = require('../../components/colorscale/defaults');
1515
var maxDimensionCount = require('./constants').maxDimensionCount;
1616

17-
var handleLineDefaults = function lineDefaults(traceIn, traceOut, defaultColor, layout, coerce) {
17+
function handleLineDefaults(traceIn, traceOut, defaultColor, layout, coerce) {
1818

1919
coerce('line.color', defaultColor);
2020
coerce('line.colorscale');
@@ -51,33 +51,21 @@ function dimensionsDefaults(traceIn, traceOut) {
5151
continue;
5252
}
5353

54-
var userVisible = coerce('visible');
55-
56-
// turn dimensions with no data invisible
57-
var actuallyVisible = userVisible && Lib.isArray(dimensionIn.values) && dimensionIn.values.length > 0;
58-
59-
if(actuallyVisible) {
54+
var values = coerce('values');
55+
var visible = coerce('visible', values.length > 0);
6056

57+
if(visible) {
6158
coerce('label');
6259
coerce('tickvals');
6360
coerce('ticktext');
6461
coerce('tickformat');
6562
coerce('range');
6663
coerce('constraintrange');
67-
coerce('values');
6864

6965
commonLength = Math.min(commonLength, dimensionOut.values.length);
70-
71-
dimensionOut._index = i;
72-
73-
} else {
74-
75-
dimensionOut = Lib.extendFlat({}, dimensionIn);
76-
7766
}
7867

79-
dimensionOut.visible = actuallyVisible;
80-
68+
dimensionOut._index = i;
8169
dimensionsOut.push(dimensionOut);
8270
}
8371

0 commit comments

Comments
 (0)