should guard againt undefined projection.invert results

... that occur in some non-clipped projections

Author: etpinard

src/plots/geo/zoom.js

@@ -114,9 +114,14 @@ function zoomNonClipped(geo, projection) {
     function position(x) { return projection.invert(x); }
 
     function outside(x) {
-        var pt = projection(position(x));
-        return (Math.abs(pt[0] - x[0]) > INSIDETOLORANCEPXS ||
-                Math.abs(pt[1] - x[1]) > INSIDETOLORANCEPXS);
+        var pos = position(x);
+        if(!pos) return true;
+
+        var pt = projection(pos);
+        return (
+            Math.abs(pt[0] - x[0]) > INSIDETOLORANCEPXS ||
+            Math.abs(pt[1] - x[1]) > INSIDETOLORANCEPXS
+        );
     }
 
     function handleZoomstart() {

test/jasmine/tests/geo_test.js

@@ -1941,4 +1941,22 @@ describe('Test geo zoom/pan/drag interactions:', function() {
         .catch(failTest)
         .then(done);
     });
+
+    it('should guard againt undefined projection.invert result in some projections', function(done) {
+        // e.g. aitoff
+        var fig = Lib.extendDeep({}, require('@mocks/geo_aitoff-sinusoidal.json'));
+        fig.layout.dragmode = 'pan';
+        delete fig.layout.geo2;
+        fig.data = [fig.data[0]];
+        fig.layout.width = 700;
+        fig.layout.height = 500;
+
+        plot(fig)
+        .then(function() { return scroll([131, 159], [-200, 200]); })
+        .then(function() {
+            // scrolling outside subplot frame should log errors,
+        })
+        .catch(failTest)
+        .then(done);
+    });
 });