New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flag tooltips do not react to hover when Boost Enabled #7646

Closed
zombiehugs opened this Issue Jan 10, 2018 · 9 comments

Comments

Projects
None yet
4 participants
@zombiehugs

zombiehugs commented Jan 10, 2018

Expected behaviour

As a user I should be able to hover over flags on series and off series to see more information.

Actual behaviour

When enabling Boost Module flags now appear on series and off series but can no longer hover over the flags for additional information. The cursor scrubs right over them.

Live demo with steps to reproduce

Run Fiddle, attempt to hover over flag.
No Hover allowed.
Hide line series and attempt to hover over flag on axis.
Flag is now responding to hover.

Minimal Fiddle: http://jsfiddle.net/gerdsen/ntjgu67n/8/

Product version

HighStock Master JS (1/10/2018)

Affected browser(s)

All

@zombiehugs zombiehugs changed the title from Boost Module, Flag Tooltips do not appear to Flag tooltips do not react to hover when Boost Enabled Jan 10, 2018

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Jan 11, 2018

Hi @zombiehugs

Thank you for reporting.

@pawelfus pawelfus added the Bug label Jan 11, 2018

@TorsteinHonsi

This comment has been minimized.

Collaborator

TorsteinHonsi commented Jan 12, 2018

Strange... At least, boost.seriesThreshold has something to do with it. When removed, the tooltip performs as expected: http://jsfiddle.net/highcharts/ntjgu67n/9/

@drichard1989

This comment has been minimized.

drichard1989 commented Jan 12, 2018

We have found that once the boostThreshold has been crossed, is when you cant hover the flags. In our application, the seriesThreshold doesn't seem to effect it.

In our app, seriesThreshold was not set to anything.

@zombiehugs

This comment has been minimized.

zombiehugs commented Jan 12, 2018

@TorsteinHonsi thanks for inspecting. So I see your fiddle is working, we tried the same within our application with little to no success. What does appear to be linked to the problem is the number of points and or series present, when we render out a small time period of say 7 days the flags work. If we tried to render a larger dataset the flags appear but cannot be hovered. This was done utilizing simple configuration and no seriesThreshold.

I updated your Fiddle from above and augmented the number of series and also the points within the series to show one working and one not.

Fiddle (working, 5 line series, 1 flag series): http://jsfiddle.net/gerdsen/ntjgu67n/10/
Fiddle (NOT working, 6 line series, 1 flag series): http://jsfiddle.net/gerdsen/ntjgu67n/11/

Thanks for all of your assistance.

@drichard1989

This comment has been minimized.

drichard1989 commented Jan 16, 2018

@TorsteinHonsi we appreciate you looking into this issue. Is there an estimated time that this issue will be resolved?

@drichard1989

This comment has been minimized.

drichard1989 commented Jan 18, 2018

@TorsteinHonsi Good morning. I wanted to follow up to see if there is an estimated time that this issue will be resolved. We have a customer that is depending on this fix for us to roll out the next patch, so we are just trying to gain some information on this timeline. We are an OEM License Holder as well. Thank you in advance.

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Jan 19, 2018

Workaround:

Set on chart's load/redraw stickyTracking flag: http://jsfiddle.net/BlackLabel/ntjgu67n/14/

function resetTracking() {
  Highcharts.each(this.series, function(series) {
    if (Highcharts.noop === series.buildKDTree) {
      series.stickyTracking = false;
    }
  })
}

Highcharts.chart('container', {
  chart: {
    events: {
      load: resetTracking,
      redraw: resetTracking
    }
  },
  ...
});
@zombiehugs

This comment has been minimized.

zombiehugs commented Jan 19, 2018

@pawelfus this seems to do the trick and we can run with this. Looking forward to seeing it in the next release! Thanks to you and the HighCharts team for the solid support.

@drichard1989

This comment has been minimized.

drichard1989 commented Jan 19, 2018

@pawelfus Thank you very much! The support here has been awesome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment