Skip to content

Commit 74070e0

Browse files
committed
refactor instanceOrPeriod function
1 parent ba48115 commit 74070e0

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

src/plots/cartesian/instance_or_period.js

+8-9
Original file line numberDiff line numberDiff line change
@@ -11,24 +11,23 @@
1111
var alignPeriod = require('./align_period');
1212

1313
module.exports = function instanceOrPeriod(d, trace, axLetter) {
14-
var ax = d[axLetter + 'a'];
14+
var ins = [
15+
d[axLetter + 'LabelVal']
16+
];
1517
if(trace[axLetter + 'period']) {
16-
var vals = [
17-
d[axLetter + 'LabelVal']
18-
];
19-
18+
var ax = d[axLetter + 'a'];
2019
var mockTrace = {};
2120
mockTrace[axLetter + 'period'] = trace[axLetter + 'period'];
2221
mockTrace[axLetter + 'period0'] = trace[axLetter + 'period0'];
2322
var alignment = axLetter + 'periodalignment';
2423

2524
mockTrace[alignment] = 'start';
26-
var startTime = alignPeriod(mockTrace, ax, axLetter, vals)[0];
25+
var startTime = alignPeriod(mockTrace, ax, axLetter, ins)[0];
2726

2827
mockTrace[alignment] = 'end';
29-
var endTime = alignPeriod(mockTrace, ax, axLetter, vals)[0];
28+
var endTime = alignPeriod(mockTrace, ax, axLetter, ins)[0];
3029

31-
return [startTime, endTime];
30+
return [startTime, endTime]; // period: two items
3231
}
33-
return [d[axLetter + 'LabelVal']];
32+
return ins; // instance: one item
3433
};

0 commit comments

Comments
 (0)