emit plotly_relayout on mapbox scroll

Author: etpinard

src/plots/mapbox/mapbox.js

@@ -130,6 +130,8 @@ proto.createMap = function(calcData, fullLayout, resolve, reject) {
 
     if(self.isStatic) return;
 
+    var wheeling = false;
+
     // keep track of pan / zoom in user layout and emit relayout event
     map.on('moveend', function(eventData) {
         if(!self.map) return;
@@ -149,11 +151,16 @@ proto.createMap = function(calcData, fullLayout, resolve, reject) {
         // mouse target (filtering out API calls) to not
         // duplicate 'plotly_relayout' events.
 
-        if(eventData.originalEvent) {
+        if(eventData.originalEvent || wheeling) {
             var update = {};
             update[self.id] = Lib.extendFlat({}, view);
             gd.emit('plotly_relayout', update);
         }
+        wheeling = false;
+    });
+
+    map.on('wheel', function() {
+        wheeling = true;
     });
 
     map.on('mousemove', function(evt) {

test/jasmine/tests/mapbox_test.js

@@ -827,6 +827,14 @@ describe('@noCI, mapbox plots', function() {
             doubleClickCnt++;
         });
 
+        function _scroll(p) {
+            return new Promise(function(resolve) {
+                mouseEvent('mousemove', p[0], p[1]);
+                mouseEvent('scroll', p[0], p[1], {deltaY: -400});
+                setTimeout(resolve, 1000);
+            });
+        }
+
         function assertLayout(center, zoom, opts) {
             var mapInfo = getMapInfo(gd),
                 layout = gd.layout.mapbox;
@@ -858,14 +866,18 @@ describe('@noCI, mapbox plots', function() {
         .then(function() {
             expect(doubleClickCnt).toBe(1, 'double click cnt');
             assertLayout([-4.710, 19.475], 1.234);
+
+            return _scroll(pointPos);
+        })
+        .then(function() {
+            expect(getMapInfo(gd).zoom).toBeGreaterThan(1.234);
+            expect(relayoutCnt).toBe(2);
         })
         .catch(failTest)
         .then(done);
-
-        // TODO test scroll
-
     }, LONG_TIMEOUT_INTERVAL);
 
+
     it('should respond to click interactions by', function(done) {
         var ptData;