Skip to content
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

Highstock 6 ignores tooltip.shared #7696

Closed
tregoning opened this issue Jan 18, 2018 · 4 comments
Closed

Highstock 6 ignores tooltip.shared #7696

tregoning opened this issue Jan 18, 2018 · 4 comments

Comments

@tregoning
Copy link

@tregoning tregoning commented Jan 18, 2018

Expected behaviour

https://api.highcharts.com/highstock/tooltip.shared should behave as Highcharts.

Actual behaviour

Highstock always behaves as if tooltip.shared was set to true.

Live demo with steps to reproduce

(Highstock) broken version: http://jsfiddle.net/tregoning/qgpLmj9y/
(Highcharts) OK version: http://jsfiddle.net/tregoning/10h23xks/

Note: that previous version of Highstock (5.0.14) does work as expected
(Highstock 5.0.14 works as expected) http://jsfiddle.net/tregoning/qts93djL/

Product version

Version with the bug: Highstock (6.0.4)

Affected browser(s)

All

@sebastianbochan
Copy link
Contributor

@sebastianbochan sebastianbochan commented Jan 19, 2018

Hi @tregoning
You should also disable the split option.

Demo:

@pawelfus
Copy link
Contributor

@pawelfus pawelfus commented Jan 19, 2018

@tregoning

Highstock always behaves as if tooltip.shared was set to true.

Let me explain. In Highstock < v6, tooltip.shared by default was set to true, now in v6 shared option is disabled as in docs:

Defaults to false.

As @sebastianbochan said, what you see now in Highstock, it's tooltip.split which by default is enabled:

In Highstock, tooltips are split by default since v6.0.0. Stock charts typically contain multi-dimension points and multiple panes, making split tooltips the preferred layout over the previous shared tooltip.

Defaults to true.

Compare:

@tregoning
Copy link
Author

@tregoning tregoning commented Jan 22, 2018

Thanks for the prompt reply. This fixed my issue.

Just heads up that it might want to consider to document the relationship/interdependency between "shared" and "split" in the documentation (where tooltip.split reads simply as an alternative).

Thanks for the great product, keep up the great work!

@pawelfus
Copy link
Contributor

@pawelfus pawelfus commented Jan 23, 2018

Thanks! You are right, it should be clear in our docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants