apply minBorderWidth only for noBorder and noFill symbols

Author: etpinard

src/traces/scattergl/convert.js

@@ -533,7 +533,7 @@ proto.updateFancy = function(options) {
 
             size = sizes[index];
             bw = borderWidths[index];
-            minBorderWidth = 0.1 * size;
+            minBorderWidth = (symbolSpec.noBorder || symbolSpec.noFill) ? 0.1 * size : 0;
 
             this.scatter.options.sizes[i] = 4.0 * size;
             this.scatter.options.glyphs[i] = symbolSpec.unicode;
@@ -544,11 +544,7 @@ proto.updateFancy = function(options) {
             // already baked into the unicode symbol, so the symbol gets no
             // explicit border until you provide a width greater than this, and
             // at greater widths you set a border borderWidth - minBorderWidth.
-
-            //this.scatter.options.borderWidths[i] = (bw > minBorderWidth) ? bw - minBorderWidth : 0;
-
-            // but I found better results with:
-            this.scatter.options.borderWidths[i] = (bw > minBorderWidth) ? bw - minBorderWidth : bw;
+            this.scatter.options.borderWidths[i] = (bw > minBorderWidth) ? bw - minBorderWidth : 0;
 
             if(isOpen && !symbolSpec.noBorder && !symbolSpec.noFill) {
                 fillColor(this.scatter.options.colors, transparent, i, 0);