Changes requested by @etpinard

Author: TomDemulierChevret

src/components/rangeslider/defaults.js

@@ -63,7 +63,7 @@ module.exports = function handleDefaults(layoutIn, layoutOut, axName) {
             if(rangeContainerIn.range && layoutOut[yName].isValidRange(rangeContainerIn.range)) {
                 coerceRange(rangeContainerIn, rangeContainerOut, 'rangemode', 'fixed');
             } else {
-                coerceRange(rangeContainerIn, rangeContainerOut, 'rangemode', 'auto');
+                coerceRange(rangeContainerIn, rangeContainerOut, 'rangemode');
             }
 
             layoutOut[yName].cleanRange('rangeslider.' + yName + '.range');

src/components/rangeslider/draw.js

@@ -430,7 +430,7 @@ function drawRangePlot(rangeSlider, gd, axisOpts, opts) {
         mockFigure.layout[oppAxisName] = {
             type: oppAxisOpts.type,
             domain: [0, 1],
-            range: oppAxisRangeOpts.range.slice(),
+            range: oppAxisRangeOpts.rangemode === 'auto' || oppAxisRangeOpts.rangemode === 'fixed' ? oppAxisRangeOpts.range.slice() : oppAxisOpts.range.slice(),
             calendar: oppAxisOpts.calendar
         };
 

src/components/rangeslider/range_attributes.js

@@ -12,7 +12,7 @@ module.exports = {
     rangemode: {
         valType: 'enumerated',
         values: ['auto', 'fixed', 'match'],
-        dflt: 'auto',
+        dflt: 'match',
         role: 'style',
         editType: 'calc',
         description: [

src/plots/cartesian/axes.js

@@ -373,12 +373,10 @@ axes.doAutoRange = function(ax) {
         if(axeRangeOpts) {
             if(axeRangeOpts.rangemode === 'auto') {
                 axeRangeOpts.range = hasDeps ? axes.getAutoRange(ax) : ax.range.slice();
-            } else {
-                axeRangeOpts.range = ax.range.slice();
             }
         }
         axIn = ax._anchorAxis._input;
-        axIn.rangeslider[ax._name] = Object.assign({}, axeRangeOpts);
+        axIn.rangeslider[ax._name] = Lib.extendFlat({}, axeRangeOpts);
     }
 };
 

test/jasmine/tests/range_slider_test.js

@@ -91,7 +91,7 @@ describe('the range slider', function() {
 
             slide(start, sliderY, end, sliderY).then(function() {
                 var maskMin = children[2],
-                    handleMin = children[7];
+                    handleMin = children[5];
 
                 expect(gd.layout.xaxis.range).toBeCloseToArray([4, 49], -0.5);
                 expect(maskMin.getAttribute('width')).toEqual(String(diff));
@@ -111,7 +111,7 @@ describe('the range slider', function() {
 
             slide(start, sliderY, end, sliderY).then(function() {
                 var maskMax = children[3],
-                    handleMax = children[8];
+                    handleMax = children[6];
 
                 expect(gd.layout.xaxis.range).toBeCloseToArray([0, 32.77], -0.5);
                 expect(+maskMax.getAttribute('width')).toBeCloseTo(-diff);
@@ -132,7 +132,7 @@ describe('the range slider', function() {
 
             slide(start, sliderY, end, sliderY).then(function() {
                 var maskMin = children[2],
-                    handleMin = children[7];
+                    handleMin = children[5];
 
                 expect(gd.layout.xaxis.range).toBeCloseToArray([3.96, 49], -0.5);
                 expect(+maskMin.getAttribute('width')).toBeCloseTo(String(diff));
@@ -152,7 +152,7 @@ describe('the range slider', function() {
 
             slide(start, sliderY, end, sliderY).then(function() {
                 var maskMax = children[3],
-                    handleMax = children[8];
+                    handleMax = children[6];
 
                 expect(gd.layout.xaxis.range).toBeCloseToArray([0, 45.04], -0.5);
                 expect(+maskMax.getAttribute('width')).toBeCloseTo(-diff);
@@ -187,8 +187,8 @@ describe('the range slider', function() {
             Plotly.relayout(gd, 'xaxis.range', [10, 20]).then(function() {
                 var maskMin = children[2],
                     maskMax = children[3],
-                    handleMin = children[7],
-                    handleMax = children[8];
+                    handleMin = children[5],
+                    handleMax = children[6];
 
                 expect(+maskMin.getAttribute('width')).toBeWithin(125, TOL);
                 expect(+maskMax.getAttribute('width')).toBeWithin(365, TOL);
@@ -203,8 +203,8 @@ describe('the range slider', function() {
             Plotly.relayout(gd, 'xaxis.range[0]', 10).then(function() {
                 var maskMin = children[2],
                     maskMax = children[3],
-                    handleMin = children[7],
-                    handleMax = children[8];
+                    handleMin = children[5],
+                    handleMax = children[6];
 
                 expect(+maskMin.getAttribute('width')).toBeWithin(126, TOL);
                 expect(+maskMax.getAttribute('width')).toEqual(0);