handle invalid range and reversed autorange

Author: archmoj

src/plots/cartesian/axis_defaults.js

@@ -103,6 +103,7 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, coerce,
     // validate range and set autorange true for invalid partial ranges
     if(range && (
         (range[0] === null && range[1] === null) ||
+        ((range[0] === null || range[1] === null) && autorange === 'reversed') ||
         (range[0] !== null && (autorange === 'min' || autorange === 'max reversed')) ||
         (range[1] !== null && (autorange === 'max' || autorange === 'min reversed'))
     )) {

src/plots/polar/layout_defaults.js

@@ -88,6 +88,7 @@ function handleDefaults(contIn, contOut, coerce, opts) {
                 // validate range and set autorange true for invalid partial ranges
                 if(range && (
                     (range[0] === null && range[1] === null) ||
+                    ((range[0] === null || range[1] === null) && autorange === 'reversed') ||
                     (range[0] !== null && (autorange === 'min' || autorange === 'max reversed')) ||
                     (range[1] !== null && (autorange === 'max' || autorange === 'min reversed'))
                 )) {

test/jasmine/tests/axes_test.js

@@ -727,9 +727,11 @@ describe('Test axes', function() {
                 yaxis5: { range: [null, 2], autorange: 'max' }, // first range is null not second
                 yaxis6: { range: [1, null], autorange: 'max reversed' }, // second range is null not first
                 yaxis7: { range: [null, 2], autorange: 'min reversed' }, // first range is null not second
+                yaxis8: { range: [1, null], autorange: 'reversed' },
+                yaxis9: { range: [null, 2], autorange: 'reversed' },
             };
-            layoutOut._subplots.cartesian.push('x2y2', 'xy3', 'x3y4', 'x3y5', 'x3y6', 'x3y7');
-            layoutOut._subplots.yaxis.push('x2', 'x3', 'y2', 'y3', 'y4', 'y5', 'y6', 'y7');
+            layoutOut._subplots.cartesian.push('x2y2', 'xy3', 'x3y4', 'x3y5', 'x3y6', 'x3y7', 'x3y9', 'x3y9');
+            layoutOut._subplots.yaxis.push('x2', 'x3', 'y2', 'y3', 'y4', 'y5', 'y6', 'y7', 'y8', 'y9');
 
             supplyLayoutDefaults(layoutIn, layoutOut, fullData);