try implementing a fast 'markerSize' editType

- marker.size is hard to edit fast, as in general it can
  change the axis ranges

Author: etpinard

src/plot_api/edit_types.js

@@ -15,7 +15,7 @@ var isPlainObject = Lib.isPlainObject;
 var traceOpts = {
     valType: 'flaglist',
     extras: ['none'],
-    flags: ['calc', 'clearAxisTypes', 'plot', 'style', 'colorbars'],
+    flags: ['calc', 'clearAxisTypes', 'plot', 'style', 'markerSize', 'colorbars'],
     description: [
         'trace attributes should include an `editType` string matching this flaglist.',
         '*calc* is the most extensive: a full `Plotly.plot` starting by clearing `gd.calcdata`',
@@ -24,7 +24,8 @@ var traceOpts = {
         'cause the automatic axis type detection to change. Log type will not be cleared, as that',
         'is never automatically chosen so must have been user-specified.',
         '*plot* calls `Plotly.plot` but without first clearing `gd.calcdata`.',
-        '*style* only calls `module.style` for all trace modules and redraws the legend.',
+        '*style* only calls `module.style` (or module.editStyle) for all trace modules and redraws the legend.',
+        '*markerSize* is like *style*, but propagate axis-range changes due to scatter `marker.size`',
         '*colorbars* only redraws colorbars.'
     ].join(' ')
 };

src/plot_api/plot_api.js

@@ -1344,8 +1344,21 @@ exports.restyle = function restyle(gd, astr, val, _traces) {
     } else {
         seq.push(Plots.previousPromises);
 
+        // maybe only call Plots.supplyDataDefaults in the splom case,
+        // to skip over long and slow axes defaults
         Plots.supplyDefaults(gd);
 
+        if(flags.markerSize) {
+            Plots.doCalcdata(gd);
+            addAxRangeSequence(seq);
+
+            // TODO
+            // if all axes have autorange:false, then
+            // proceed to subroutines.doTraceStyle(),
+            // otherwise we must go through addAxRangeSequence,
+            // which in general must redraws 'all' axes
+        }
+
         if(flags.style) seq.push(subroutines.doTraceStyle);
         if(flags.colorbars) seq.push(subroutines.doColorBars);
 

src/traces/splom/attributes.js

@@ -104,7 +104,7 @@ module.exports = {
 
     marker: extendFlat({}, colorAttrs('marker'), {
         symbol: scatterMarkerAttrs.symbol,
-        size: scatterMarkerAttrs.size,
+        size: extendFlat({}, scatterMarkerAttrs.size, {editType: 'markerSize'}),
         sizeref: scatterMarkerAttrs.sizeref,
         sizemin: scatterMarkerAttrs.sizemin,
         sizemode: scatterMarkerAttrs.sizemode,

test/jasmine/tests/splom_test.js

@@ -922,6 +922,40 @@ describe('Test splom update switchboard:', function() {
 
             return Plotly.restyle(gd, 'marker.size', 20);
         })
+        .then(function() {
+            var msg = 'after scalar marker.size restyle';
+
+            assertSpies(msg, [
+                ['supplyDefaults', 1],
+                ['doCalcdata', 1],
+                ['doTicks', 1],
+                ['regl clear', 1],
+                ['update', 1],
+                ['draw', 1]
+            ]);
+
+            expect(scene.matrixOptions.size).toBe(10, msg);
+            expect(gd._fullLayout.xaxis.range)
+                .toBeCloseToArray([0.753, 3.246], 1, 'xrng ' + msg);
+
+            return Plotly.restyle(gd, 'marker.size', [[4, 10, 20]]);
+        })
+        .then(function() {
+            var msg = 'after scalar marker.size restyle';
+
+            assertSpies(msg, [
+                ['supplyDefaults', 1],
+                ['doCalcdata', 1],
+                ['doTicks', 1],
+                ['regl clear', 1],
+                ['update', 1],
+                ['draw', 1]
+            ]);
+
+            expect(scene.matrixOptions.sizes).toBeCloseToArray([2, 5, 10], 1, msg);
+            expect(gd._fullLayout.xaxis.range)
+                .toBeCloseToArray([0.853, 3.235], 1, 'xrng ' + msg);
+        })
         .catch(failTest)
         .then(done);
     });