adapt Histogram.eventData so that it works with selection pts

Author: etpinard

src/traces/histogram/event_data.js

@@ -6,21 +6,23 @@
 * LICENSE file in the root directory of this source tree.
 */
 
-
 'use strict';
 
-
-module.exports = function eventData(out, pt) {
+module.exports = function eventData(out, pt, trace, cd, pointNumber) {
     // standard cartesian event data
-    out.x = pt.xVal;
-    out.y = pt.yVal;
-    out.xaxis = pt.xa;
-    out.yaxis = pt.ya;
+    out.x = 'xVal' in pt ? pt.xVal : pt.x;
+    out.y = 'yVal' in pt ? pt.yVal : pt.y;
+
+    if(pt.xa) out.xaxis = pt.xa;
+    if(pt.ya) out.yaxis = pt.ya;
+
+    // specific to histogram - CDFs do not have pts (yet?)
+    if(!(trace.cumulative || {}).enabled) {
+        var pts = Array.isArray(pointNumber) ?
+            cd[0].pts[pointNumber[0]][pointNumber[1]] :
+            cd[pointNumber].pts;
 
-    // specific to histogram
-    // CDFs do not have pts (yet?)
-    if(pt.pts) {
-        out.pointNumbers = pt.pts;
+        out.pointNumbers = pts;
         out.binNumber = out.pointNumber;
         delete out.pointNumber;
     }

src/traces/histogram/hover.js

@@ -25,7 +25,6 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
         var posLetter = trace.orientation === 'h' ? 'y' : 'x';
 
         pointData[posLetter + 'Label'] = hoverLabelText(pointData[posLetter + 'a'], di.p0, di.p1);
-        pointData.pts = di.pts;
     }
 
     return pts;

src/traces/histogram2d/hover.js

@@ -27,7 +27,6 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode, hoverLay
 
     pointData.xLabel = hoverLabelText(pointData.xa, xRange[0], xRange[1]);
     pointData.yLabel = hoverLabelText(pointData.ya, yRange[0], yRange[1]);
-    pointData.pts = cd0.pts[ny][nx];
 
     return pts;
 };