provide bbox for sunburst treemap and icicle

Author: archmoj

src/traces/sunburst/fx.js

@@ -52,6 +52,8 @@ module.exports = function attachFxHandlers(sliceTop, entry, gd, cd, opts) {
         var hoverinfo = Fx.castHoverinfo(traceNow, fullLayoutNow, ptNumber);
         var separators = fullLayoutNow.separators;
 
+        var eventData;
+
         if(hovertemplate || (hoverinfo && hoverinfo !== 'none' && hoverinfo !== 'skip')) {
             var hoverCenterX;
             var hoverCenterY;
@@ -125,6 +127,8 @@ module.exports = function attachFxHandlers(sliceTop, entry, gd, cd, opts) {
                 if(Lib.isValidTextValue(tx)) thisText.push(tx);
             }
 
+            eventData = [makeEventData(pt, traceNow, opts.eventDataKeys)];
+
             var hoverItems = {
                 trace: traceNow,
                 y: hoverCenterY,
@@ -139,7 +143,7 @@ module.exports = function attachFxHandlers(sliceTop, entry, gd, cd, opts) {
                 textAlign: _cast('hoverlabel.align'),
                 hovertemplate: hovertemplate,
                 hovertemplateLabels: hoverPt,
-                eventData: [makeEventData(pt, traceNow, opts.eventDataKeys)]
+                eventData: eventData
             };
 
             if(isSunburst) {
@@ -152,11 +156,14 @@ module.exports = function attachFxHandlers(sliceTop, entry, gd, cd, opts) {
                 hoverItems.idealAlign = hoverCenterX < 0 ? 'left' : 'right';
             }
 
+            var bbox = [];
             Fx.loneHover(hoverItems, {
                 container: fullLayoutNow._hoverlayer.node(),
                 outerContainer: fullLayoutNow._paper.node(),
-                gd: gd
+                gd: gd,
+                inOut_bbox: bbox
             });
+            eventData[0].bbox = bbox[0];
 
             trace._hasHoverLabel = true;
         }
@@ -170,7 +177,7 @@ module.exports = function attachFxHandlers(sliceTop, entry, gd, cd, opts) {
 
         trace._hasHoverEvent = true;
         gd.emit('plotly_hover', {
-            points: [makeEventData(pt, traceNow, opts.eventDataKeys)],
+            points: eventData || [makeEventData(pt, traceNow, opts.eventDataKeys)],
             event: d3.event
         });
     };

test/jasmine/tests/icicle_test.js

@@ -526,6 +526,12 @@ describe('Test icicle hover:', function() {
                     expect(ptData[k]).toBe(exp.ptData[k], 'pt event data key ' + k);
                 }
 
+                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');
+
                 if(exp.style) {
                     var gd3 = d3Select(gd);
                     assertHoverLabelStyle(gd3.select('.hovertext'), exp.style);

test/jasmine/tests/sunburst_test.js

@@ -539,6 +539,12 @@ describe('Test sunburst hover:', function() {
                     expect(ptData[k]).toBe(exp.ptData[k], 'pt event data key ' + k);
                 }
 
+                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');
+
                 if(exp.style) {
                     var gd3 = d3Select(gd);
                     assertHoverLabelStyle(gd3.select('.hovertext'), exp.style);

test/jasmine/tests/treemap_test.js

@@ -628,6 +628,12 @@ describe('Test treemap hover:', function() {
                     expect(ptData[k]).toBe(exp.ptData[k], 'pt event data key ' + k);
                 }
 
+                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');
+
                 if(exp.style) {
                     var gd3 = d3Select(gd);
                     assertHoverLabelStyle(gd3.select('.hovertext'), exp.style);