do not scale scattergl line.width by marker.sizeref

etpinard · etpinard · commit 9055d49c2932 · 2018-05-28T18:08:13.000-04:00
diff --git a/src/traces/scattergl/convert.js b/src/traces/scattergl/convert.js
@@ -192,10 +192,10 @@ function convertMarkerStyle(trace) {
         // See  https://github.com/plotly/plotly.js/pull/1781#discussion_r121820798
         if(multiLineWidth) {
             for(i = 0; i < count; i++) {
-                borderSizes[i] = markerSizeFunc(optsIn.line.width[i]);
+                borderSizes[i] = optsIn.line.width[i] / 2;
             }
         } else {
-            s = markerSizeFunc(optsIn.line.width);
+            s = optsIn.line.width / 2;
             for(i = 0; i < count; i++) {
                 borderSizes[i] = s;
             }

Original file line number	Diff line number	Diff line change
`@@ -192,10 +192,10 @@ function convertMarkerStyle(trace) {`
`192`	`192`	`// See https://github.com/plotly/plotly.js/pull/1781#discussion_r121820798`
`193`	`193`	`if(multiLineWidth) {`
`194`	`194`	`for(i = 0; i < count; i++) {`
`195`		`- borderSizes[i] = markerSizeFunc(optsIn.line.width[i]);`
	`195`	`+ borderSizes[i] = optsIn.line.width[i] / 2;`
`196`	`196`	`}`
`197`	`197`	`} else {`
`198`		`- s = markerSizeFunc(optsIn.line.width);`
	`198`	`+ s = optsIn.line.width / 2;`
`199`	`199`	`for(i = 0; i < count; i++) {`
`200`	`200`	`borderSizes[i] = s;`
`201`	`201`	`}`