Endzone Tooltips fix for FF.

[panda01]

Closes #3930.

Touches all of the tooltips.

[Tooltip] Changed the way the tooltip functions to make up for the fact firefox doesn't have certain properties on the event and event target.

firefox's event is missing the offsetX which gives based on the nearest relative positioned element.

[spalger]

Did you mean to get rid of the xScale() stuff here?

[panda01]

I did. It doesn't seem to do anything. especially considering the xLeft and such seems to just be used to determine the position of the mouse.

[spalger]

That code is what's responsible for showing the warning anytime you hover on a bucket that goes out of bounds.

[panda01]

Idk man it does that now. It seemed to me this is what the function was
really trying to accomplish
On May 26, 2015 4:43 PM, "Spencer" notifications@github.com wrote:

In src/kibana/components/vislib/visualizations/_point_series_chart.js
#3980 (comment):

     var min = leftEndzone.w;
     var max = rightEndzone.x;

     // bounds of the cursor to consider

•    var xLeft = event.offsetX;
  

•    var xRight = event.offsetX;
  

•    if (wholeBucket) {
  

•      xLeft = xScale(boundData.x);
  

That code is what's responsible for showing the warning anytime you hover
on a bucket that goes out of bounds.

—
Reply to this email directly or view it on GitHub
https://github.com/elastic/kibana/pull/3980/files#r31077611.

[spalger]

This is what I'm talking about:

[spalger]

and on master:

[panda01]

Addressed @spalger's concerns.

[spalger]

LGTM!