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

Tooltip doesn't update after hovering above / below points (over plot area) [5.0.8+] #6456

Closed
stpoa opened this Issue Mar 12, 2017 · 5 comments

Comments

Projects
None yet
5 participants
@stpoa

stpoa commented Mar 12, 2017

Expected behaviour

After hovering over point on series and then hovering above or below different point, tooltip should update its position. (Works as expected in 5.0.7)

Actual behaviour

After hovering over point on series and then hovering above or below different point, tooltip doesn't update its position.

Live demo with steps to reproduce

http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/stock/demo/intraday-candlestick/

  1. Hover over one point
  2. Move your mouse cursor above or below point on either right or left side
  3. Notice tooltip doesn't redraw
@onmas59

This comment has been minimized.

Show comment
Hide comment
@onmas59

onmas59 Mar 13, 2017

Hi Torsten,

It was me that filed this bug originally with stpoa. I have a workaround (using 5.0.7) and wondered when you might expect to provide an official patch for 5.0.8 please?
Thanks.
Jason

onmas59 commented Mar 13, 2017

Hi Torsten,

It was me that filed this bug originally with stpoa. I have a workaround (using 5.0.7) and wondered when you might expect to provide an official patch for 5.0.8 please?
Thanks.
Jason

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard Mar 15, 2017

Collaborator

Hi @onmas59
We will have this fixed by the next maintenance release, which is likely to be released in a few weeks.
In the meantime you could try out the workaround below.

Workaround
Example of the workaround
Add the following snippet after loading Highcharts and before initializing the chart.

Highcharts.wrap(Highcharts.Pointer.prototype, 'getHoverData', function (proceed, a, b, c, isDirectTouch, shared, f) {
    var directTouch = shared ? false : directTouch;
    return proceed.apply(this, [a, b, c, directTouch, shared, f]);
});
Collaborator

jon-a-nygaard commented Mar 15, 2017

Hi @onmas59
We will have this fixed by the next maintenance release, which is likely to be released in a few weeks.
In the meantime you could try out the workaround below.

Workaround
Example of the workaround
Add the following snippet after loading Highcharts and before initializing the chart.

Highcharts.wrap(Highcharts.Pointer.prototype, 'getHoverData', function (proceed, a, b, c, isDirectTouch, shared, f) {
    var directTouch = shared ? false : directTouch;
    return proceed.apply(this, [a, b, c, directTouch, shared, f]);
});
@onmas59

This comment has been minimized.

Show comment
Hide comment
@onmas59

onmas59 Mar 15, 2017

Hi @jon-a-nygaard

Thanks for the update and workaround.

This fixes it for the axis used by the candle data series, but I have other axes (e.g. showing Volume, technical indicators etc.), which fail to show on the tooltip. The workaround I'm using (going back to using 5.0.7) works fine for the moment. I'll keep an eye out for the next release in a few weeks.

Thanks again.
Jason

onmas59 commented Mar 15, 2017

Hi @jon-a-nygaard

Thanks for the update and workaround.

This fixes it for the axis used by the candle data series, but I have other axes (e.g. showing Volume, technical indicators etc.), which fail to show on the tooltip. The workaround I'm using (going back to using 5.0.7) works fine for the moment. I'll keep an eye out for the next release in a few weeks.

Thanks again.
Jason

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Mar 20, 2017

Contributor

Internal note:

@oysteinmoseng @jon-a-nygaard
This fix will break accessibility module.
Demo: http://jsfiddle.net/sd6b3c6n/1/
Steps:

  1. click under the chart
  2. press tab
  3. press right arrow
  4. error in the console - calling for a shared tooltip without an event
Contributor

KacperMadej commented Mar 20, 2017

Internal note:

@oysteinmoseng @jon-a-nygaard
This fix will break accessibility module.
Demo: http://jsfiddle.net/sd6b3c6n/1/
Steps:

  1. click under the chart
  2. press tab
  3. press right arrow
  4. error in the console - calling for a shared tooltip without an event
@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard May 22, 2017

Collaborator

@KacperMadej See comment in #6633

Collaborator

jon-a-nygaard commented May 22, 2017

@KacperMadej See comment in #6633

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