GL rangeslider saftey check

mdtusz · mdtusz · commit 948f493c8542 · 2016-03-31T15:03:26.000-04:00
diff --git a/src/components/rangeslider/create_slider.js b/src/components/rangeslider/create_slider.js
@@ -249,8 +249,8 @@ module.exports = function createSlider(gd, minStart, maxStart) {
     ]);
 
     sliderContainer.data([0])
-    .enter().append(function() {
-        options.setRange = setRange;
-        return slider;
-    });
+        .enter().append(function() {
+            options.setRange = setRange;
+            return slider;
+        });
 };
diff --git a/src/components/rangeslider/index.js b/src/components/rangeslider/index.js
@@ -41,11 +41,15 @@ function draw(gd, minStart, maxStart) {
     var height = (fullLayout.height - fullLayout.margin.b - fullLayout.margin.t) * options.thickness,
         offsetShift = Math.floor(options.borderwidth / 2);
 
-    if(sliderContainer[0].length === 0) createSlider(gd, minStart, maxStart);
+    if(sliderContainer[0].length === 0 && !fullLayout._hasGL2D) createSlider(gd, minStart, maxStart);
+
+    // Need to default to 0 for when making gl plots
+    var bb = fullLayout.xaxis._boundingBox ?
+        fullLayout.xaxis._boundingBox.height : 0;
 
     Plots.autoMargin(gd, 'range-slider', {
         x: 0, y: 0, l: 0, r: 0, t: 0,
-        b: height + fullLayout.margin.b + fullLayout.xaxis._boundingBox.height,
+        b: height + fullLayout.margin.b + bb,
         pad: 15 + offsetShift * 2
     });
 }
diff --git a/test/jasmine/tests/range_slider_test.js b/test/jasmine/tests/range_slider_test.js
@@ -308,6 +308,29 @@ describe('the range slider', function() {
             expect(layoutOut).toEqual(expected);
         });
     });
+
+    fdescribe('when used with gl traces', function() {
+
+        beforeEach(function() {
+            gd = createGraphDiv();
+        });
+
+        afterEach(destroyGraphDiv);
+
+        it('should not be drawn with GL2d', function(done) {
+            var data = [{
+                x: [1,2,3],
+                y: [2,3,4],
+                type: 'scattergl'
+            }];
+
+            Plotly.plot(gd, data, mock.layout).then(function() {
+                rangeSlider = document.getElementsByClassName('range-slider')[0];
+                expect(rangeSlider).not.toBeDefined();
+                done();
+            });
+        });
+    });
 });
 
 
