Skip to content

Commit bcf7b27

Browse files
committed
eslint: add / enforce curly: 'multi-line' rule
- this was only violated once in drawing (fixed in this commit)
1 parent f394d4f commit bcf7b27

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

.eslintrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"indent": [2, 4, {"SwitchCase": 1}],
1818
"max-len": [0, 80],
1919
"brace-style": [0, "stroustrup", {"allowSingleLine": true}],
20-
"curly": [0, "multi"],
20+
"curly": [2, "multi-line"],
2121
"camelcase": [0, {"properties": "never"}],
2222
"comma-spacing": [2, {"before": false, "after": true}],
2323
"comma-style": [2, "last"],

src/components/drawing/index.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,8 +204,10 @@ function singlePointStyle(d, sel, trace, markerScale, lineScale, marker, markerL
204204

205205
// handle multi-trace graph edit case
206206
if(d.ms === 'various' || marker.size === 'various') r = 3;
207-
else r = subTypes.isBubble(trace) ?
207+
else {
208+
r = subTypes.isBubble(trace) ?
208209
sizeFn(d.ms) : (marker.size || 6) / 2;
210+
}
209211

210212
// store the calculated size so hover can use it
211213
d.mrc = r;

0 commit comments

Comments
 (0)