provide bbox in gl3d event data

Author: archmoj

src/components/fx/hover.js

@@ -171,6 +171,10 @@ exports.loneHover = function loneHover(hoverItems, opts) {
                 y0: y0 + gTop,
                 y1: y1 + gTop
             };
+
+            if(opts.inOut_bbox) {
+                opts.inOut_bbox.push(eventData.bbox);
+            }
         } else {
             eventData = false;
         }

src/plots/gl3d/scene.js

@@ -421,6 +421,7 @@ proto.render = function() {
         var eventData = {points: [pointData]};
 
         if(scene.fullSceneLayout.hovermode) {
+            var bbox = [];
             Fx.loneHover({
                 trace: traceNow,
                 x: (0.5 + 0.5 * pdata[0] / pdata[3]) * width,
@@ -442,8 +443,11 @@ proto.render = function() {
                 eventData: [pointData]
             }, {
                 container: svgContainer,
-                gd: gd
+                gd: gd,
+                inOut_bbox: bbox
             });
+
+            pointData.bbox = bbox[0];
         }
 
         if(selection.buttons && selection.distance < 5) {

test/jasmine/tests/gl3d_hover_click_test.js

@@ -58,7 +58,8 @@ describe('Test gl3d trace click/hover:', function() {
     function assertEventData(x, y, z, curveNumber, pointNumber, extra) {
         expect(Object.keys(ptData)).toEqual(jasmine.arrayContaining([
             'x', 'y', 'z',
-            'data', 'fullData', 'curveNumber', 'pointNumber'
+            'data', 'fullData', 'curveNumber', 'pointNumber',
+            'bbox'
         ]), 'correct hover data fields');
 
         expect(ptData.x).toEqual(x, 'x val');
@@ -67,6 +68,12 @@ describe('Test gl3d trace click/hover:', function() {
         expect(ptData.curveNumber).toEqual(curveNumber, 'curveNumber');
         expect(ptData.pointNumber).toEqual(pointNumber, 'pointNumber');
 
+        expect(typeof ptData.bbox).toEqual('object');
+        expect(typeof ptData.bbox.x0).toEqual('number');
+        expect(typeof ptData.bbox.x1).toEqual('number');
+        expect(typeof ptData.bbox.y0).toEqual('number');
+        expect(typeof ptData.bbox.y1).toEqual('number');
+
         Object.keys(extra || {}).forEach(function(k) {
             expect(ptData[k]).toEqual(extra[k], k + ' val');
         });

test/jasmine/tests/streamtube_test.js

@@ -593,7 +593,7 @@ describe('Test streamtube hover', function() {
         .then(delay(20))
         .then(function() {
             if(ptData) {
-                expect(Object.keys(ptData).length).toBe(12, 'key cnt');
+                expect(Object.keys(ptData).length).toBe(13, 'key cnt');
 
                 expect(ptData.tubex).toBeCloseTo(2.19, TOL, 'tubex');
                 expect(ptData.tubey).toBeCloseTo(0.55, TOL, 'tubey');