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 useHTML=true not working on HighStock in 6.0.1 #7238

Closed
primerano opened this Issue Oct 6, 2017 · 7 comments

Comments

Projects
None yet
5 participants
@primerano

primerano commented Oct 6, 2017

Expected behaviour

useHtml: true should result in a html tooltip

Actual behaviour

SVG is always returned on highstock 6.0.1

Live demo with steps to reproduce

HTML tooltip works in 5.x

http://jsfiddle.net/primerano/0grm1vro/

HTML tooltip does not work in 6.x

http://jsfiddle.net/primerano/Lf4qyt8t/

Affected browser(s)

all

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Oct 6, 2017

Contributor

Hi @primerano

This is caused by change in v6: in v6 tooltip.split is set to true by default, without useHTML support. You can get previous tooltip with useHTML by disabling split, demo: http://jsfiddle.net/0grm1vro/2/

Contributor

pawelfus commented Oct 6, 2017

Hi @primerano

This is caused by change in v6: in v6 tooltip.split is set to true by default, without useHTML support. You can get previous tooltip with useHTML by disabling split, demo: http://jsfiddle.net/0grm1vro/2/

@bazineta

This comment has been minimized.

Show comment
Hide comment
@bazineta

bazineta Oct 9, 2017

While at least in our usage, setting setting split: false does rectify the issue, the documentation indicates that the default is false:

http://api.highcharts.com/highcharts/tooltip.split

Is this a temporary bug that we need to workaround for the moment, or a permanent change? If it's permanent, then the documentation for useHTML should also be modified to indicate the requirement.

bazineta commented Oct 9, 2017

While at least in our usage, setting setting split: false does rectify the issue, the documentation indicates that the default is false:

http://api.highcharts.com/highcharts/tooltip.split

Is this a temporary bug that we need to workaround for the moment, or a permanent change? If it's permanent, then the documentation for useHTML should also be modified to indicate the requirement.

@primerano

This comment has been minimized.

Show comment
Hide comment
@primerano

primerano Oct 10, 2017

Setting this to false fixed my problem. Thanks.

It seems that this default of true only applies to highstock and not highmaps or the base highcharts.

Perhaps the default should be put back to false for consistency?

primerano commented Oct 10, 2017

Setting this to false fixed my problem. Thanks.

It seems that this default of true only applies to highstock and not highmaps or the base highcharts.

Perhaps the default should be put back to false for consistency?

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Oct 10, 2017

Contributor

This is expected change, just docs are not updated, we will fix that, thank you for letting us know!

@TorsteinHonsi - what do you think about split + useHTML?

Contributor

pawelfus commented Oct 10, 2017

This is expected change, just docs are not updated, we will fix that, thank you for letting us know!

@TorsteinHonsi - what do you think about split + useHTML?

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Oct 17, 2017

Collaborator

the documentation indicates that the default is false

You are linking to the Highcharts documentation. Highstock has a different default: http://api.highcharts.com/highstock/tooltip.split

Under any circumstance, useHTML is fixed now.

Collaborator

TorsteinHonsi commented Oct 17, 2017

the documentation indicates that the default is false

You are linking to the Highcharts documentation. Highstock has a different default: http://api.highcharts.com/highstock/tooltip.split

Under any circumstance, useHTML is fixed now.

@anthonyjlmorel

This comment has been minimized.

Show comment
Hide comment
@anthonyjlmorel

anthonyjlmorel Oct 18, 2017

I just ran into this issue and just saw you provided a fix. Do you have an idea of when are you gonna release this (in a 6.0.2 for instance) ?

anthonyjlmorel commented Oct 18, 2017

I just ran into this issue and just saw you provided a fix. Do you have an idea of when are you gonna release this (in a 6.0.2 for instance) ?

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Oct 18, 2017

Contributor

Hi, v6.0.2 should be released this week.

Contributor

pawelfus commented Oct 18, 2017

Hi, v6.0.2 should be released this week.

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