axRange edits don't need drawMarginPushers after all

must have been necessary mid-development then fixed later on?

Author: alexcjohnson

src/plot_api/plot_api.js

@@ -1732,8 +1732,7 @@ function addAxRangeSequence(seq, rangesAltered) {
     seq.push(
         doTicks,
         subroutines.drawData,
-        subroutines.finalDraw,
-        subroutines.drawMarginPushers
+        subroutines.finalDraw
     );
 }
 

test/jasmine/tests/axes_test.js

@@ -2655,6 +2655,40 @@ describe('Test axes', function() {
 
                 previousSize = Lib.extendDeep({}, size);
                 savedBottom = previousSize.b;
+
+                // move all the long x labels off-screen
+                return Plotly.relayout(gd, {'xaxis.range': [-10, -5]});
+            })
+            .then(function() {
+                var size = gd._fullLayout._size;
+                expect(size.l).toBe(previousSize.l);
+                expect(size.r).toBe(previousSize.r);
+                expect(size.t).toBe(previousSize.t);
+                expect(size.b).toBe(initialSize.b);
+
+                // move all the long y labels off-screen
+                return Plotly.relayout(gd, {'yaxis.range': [-10, -5]});
+            })
+            .then(function() {
+                var size = gd._fullLayout._size;
+                expect(size.l).toBe(initialSize.l);
+                expect(size.r).toBe(previousSize.r);
+                expect(size.t).toBe(previousSize.t);
+                expect(size.b).toBe(initialSize.b);
+
+                // bring the long labels back
+                return Plotly.relayout(gd, {
+                    'xaxis.autorange': true,
+                    'yaxis.autorange': true
+                });
+            })
+            .then(function() {
+                var size = gd._fullLayout._size;
+                expect(size.l).toBe(previousSize.l);
+                expect(size.r).toBe(previousSize.r);
+                expect(size.t).toBe(previousSize.t);
+                expect(size.b).toBe(previousSize.b);
+
                 return Plotly.relayout(gd, {'xaxis.tickangle': 45});
             })
             .then(function() {

test/jasmine/tests/plot_api_test.js

@@ -632,7 +632,7 @@ describe('Test plot api', function() {
         });
 
         it('should trigger minimal sequence for cartesian axis range updates', function() {
-            var seq = ['doTicksRelayout', 'drawData', 'finalDraw', 'drawMarginPushers'];
+            var seq = ['doTicksRelayout', 'drawData', 'finalDraw'];
 
             function _assert(msg) {
                 expect(gd.calcdata).toBeDefined();