clear polar radial range viewInitial during log <-> linear relayout

etpinard · etpinard · commit 53575019534a · 2018-01-10T18:26:57.000-05:00
diff --git a/src/plot_api/plot_api.js b/src/plot_api/plot_api.js
@@ -1944,6 +1944,12 @@ function _relayout(gd, aobj) {
                     ax.range = (ax.range[1] > ax.range[0]) ? [1, 2] : [2, 1];
                 }
 
+                // clear polar view initial stash for radial range so that
+                // value get recomputed in correct units
+                if(fullLayout._subplots.polar && fullLayout[p.parts[0]]) {
+                    delete fullLayout[p.parts[0]]._subplot.viewInitial['radialaxis.range'];
+                }
+
                 // Annotations and images also need to convert to/from linearized coords
                 // Shapes do not need this :)
                 Registry.getComponentMethod('annotations', 'convertCoords')(gd, parentFull, vi, doextra);
diff --git a/test/jasmine/tests/polar_test.js b/test/jasmine/tests/polar_test.js
@@ -232,8 +232,21 @@ describe('Test relayout on polar subplots:', function() {
         var fig = Lib.extendDeep({}, require('@mocks/polar_scatter.json'));
 
         Plotly.plot(gd, fig).then(function() {
+            expect(gd._fullLayout.polar._subplot.viewInitial['radialaxis.range'])
+                .toBeCloseToArray([0, 11.225]);
+
             return Plotly.relayout(gd, 'polar.radialaxis.type', 'log');
         })
+        .then(function() {
+            expect(gd._fullLayout.polar._subplot.viewInitial['radialaxis.range'])
+                .toBeCloseToArray([-0.53, 1.158]);
+
+            return Plotly.relayout(gd, 'polar.radialaxis.type', 'linear');
+        })
+        .then(function() {
+            expect(gd._fullLayout.polar._subplot.viewInitial['radialaxis.range'])
+                .toBeCloseToArray([0, 11.225]);
+        })
         .catch(fail)
         .then(done);
     });
