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

HighMaps - mappoint Series Tooltip Ignores numberFormat #19465

Closed
wergeld opened this issue Jul 28, 2023 · 3 comments · Fixed by #19484
Closed

HighMaps - mappoint Series Tooltip Ignores numberFormat #19465

wergeld opened this issue Jul 28, 2023 · 3 comments · Fixed by #19484

Comments

@wergeld
Copy link

wergeld commented Jul 28, 2023

Expected behaviour

Tooltip should obey numberFormat settings for mappoint series.

Actual behaviour

Tooltip does not obey numberFormat settingsmappoint series.

Live demo with steps to reproduce

https://jsfiddle.net/wergeld/pyuhazs1/5/

Hover over the top-left most map marker for Reykjavik. The tooltip should show "10,000.000".
I have added setOptions and also explicitly set the the Highcharts.numberFormat on the tooltip formatter. In either case the desire to use the "," as the thousands separator is ignored.

This does work as expected for other chart series, just not mappoint.

Product version

HighMaps v11.1

Affected browser(s)

Chrome, Firefox, Edge.

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Jul 28, 2023
@bm64
Copy link
Member

bm64 commented Aug 1, 2023

Hello @wergeld and thanks for contacting us.

I think there may have been a slight mix-up with the property name. You used thousandsSeparator, but it should be thousandsSep. When using thousandsSep, it seems to work correctly. Please check out this demo: https://jsfiddle.net/BlackLabel/4ow35cad/

API reference: https://api.highcharts.com/highcharts/lang.thousandsSep

I'm closing the ticket, but if the issue persists feel free to reopen it

@bm64 bm64 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 1, 2023
Development-Flow automation moved this from To do to Done Aug 1, 2023
@wergeld
Copy link
Author

wergeld commented Aug 1, 2023

Interesting. I took the "thousandsSeparator" directly from an example jsFiddle (https://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/members/highcharts-numberformat) from the API docs (https://api.highcharts.com/highcharts/chart.numberFormatter).

That document/example should be updated.

@bm64
Copy link
Member

bm64 commented Aug 1, 2023

@wergeld thanks for pointing it out, we will take care about this

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

Successfully merging a pull request may close this issue.

4 participants