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

Tooltip formatting options don't work for scatter chart #907

Closed
pawelfus opened this issue Apr 19, 2012 · 5 comments
Closed

Tooltip formatting options don't work for scatter chart #907

pawelfus opened this issue Apr 19, 2012 · 5 comments

Comments

@pawelfus
Copy link
Contributor

For other types it works fine, but for scatter doesn't respect headerFormat and pointFormat: http://jsfiddle.net/Fusher/Sq3KL/1/

@TorsteinHonsi
Copy link
Collaborator

The problem is that scatter has its own pointFormat and headerFormat defined in plotOptions.scatter.tooltip. So we need to override this: http://jsfiddle.net/highcharts/Sq3KL/2/.

We need to find a smarter way to check whether the tooltip option has been altered.

@pawelfus
Copy link
Contributor Author

The same applies for an arearange chart: http://jsfiddle.net/M6PLJ/7/

@TorsteinHonsi
Copy link
Collaborator

... and for columnrange

@ywen
Copy link

ywen commented Apr 22, 2013

So will this bug be fixed?

@TorsteinHonsi
Copy link
Collaborator

Proposed solution: In Series.js, in the setOptions method, set the series.tooltipOptions like so:

        // the tooltip options are merged between global and series specific options
        this.tooltipOptions = merge(
            defaultOptions.tooltip,
            typeOptions.tooltip,
            chartOptions.tooltip,
            plotOptions.series && plotOptions.series.tooltip,
            itemOptions.tooltip
        );

It still feels like a bloat, since we're doing similar things with options in general...

nrcbwelch pushed a commit to NationalResearchCorporation/highcharts.com that referenced this issue Feb 18, 2014
…types where the same options were defined in series specific defaults. Closes highcharts#907.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants