choose more heavy duty patch for setting mode: 'markers' in splom traces

Author: etpinard

src/components/legend/style.js

@@ -83,7 +83,7 @@ module.exports = function style(s, gd) {
     function stylePoints(d) {
         var d0 = d[0],
             trace = d0.trace,
-            showMarkers = subTypes.hasMarkers(trace) || trace.type === 'splom',
+            showMarkers = subTypes.hasMarkers(trace),
             showText = subTypes.hasText(trace),
             showLines = subTypes.hasLines(trace);
 

src/traces/scatter/subtypes.js

@@ -18,8 +18,11 @@ module.exports = {
     },
 
     hasMarkers: function(trace) {
-        return trace.visible && trace.mode &&
-            trace.mode.indexOf('markers') !== -1;
+        return trace.visible && (
+            (trace.mode && trace.mode.indexOf('markers') !== -1) ||
+            // until splom implement 'mode'
+            trace.type === 'splom'
+        );
     },
 
     hasText: function(trace) {

src/traces/splom/index.js

@@ -43,12 +43,8 @@ function calc(gd, trace) {
         }
     }
 
-    // add 'mode' to splom trace object to reuse scatter markers logic,
-    // but do not mutate fullData items!
-    var scatterTrace = Lib.extendFlat({}, trace, {mode: 'markers'});
-
-    calcColorscales(scatterTrace);
-    Lib.extendFlat(opts, convertMarkerStyle(scatterTrace));
+    calcColorscales(trace);
+    Lib.extendFlat(opts, convertMarkerStyle(trace));
 
     var visibleLength = matrixData.length;
     var hasTooManyPoints = (visibleLength * commonLength) > TOO_MANY_POINTS;
@@ -69,10 +65,10 @@ function calc(gd, trace) {
             if(hasTooManyPoints) {
                 ppad = 2 * (opts.sizeAvg || Math.max(opts.size, 3));
             } else {
-                ppad = calcMarkerSize(scatterTrace, commonLength);
+                ppad = calcMarkerSize(trace, commonLength);
             }
 
-            calcAxisExpansion(gd, scatterTrace, xa, ya, matrixData[k], matrixData[k], ppad);
+            calcAxisExpansion(gd, trace, xa, ya, matrixData[k], matrixData[k], ppad);
             k++;
         }
     }