Right now when displaying large numbers, we do not include a thousands separator (typically a comma) in the display of the chart. We should include a comma by default, and if possible allow this to be localized.

We're already formatting numbers to support decimal places, prefixes, and suffixes, so we only need to modify the existing formatting to support thousands separators.

See http://docs.highcharts.com/#formatting
https://developers.google.com/chart/interactive/docs/reference#formatters

CommentFileSizeAuthor
#6 highcharts.js_.txt188.18 KBkellyimagined
#3 google_charts_large-2418717-1.patch826 byteshomberger
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pierre.Vriens’s picture

Pierre.Vriens CreditAttribution: Pierre.Vriens commented
Parent issue: » #2395779: Charts 7.x-3.x Release
Pierre.Vriens’s picture

Pierre.Vriens CreditAttribution: Pierre.Vriens commented
Status: Active » Needs work
Parent issue: #2395779: Charts 7.x-3.x Release » #2382151: Charts 7.x-2.0 Release
homberger’s picture

homberger CreditAttribution: homberger as a volunteer commented
Title: Include thousands separator in chart display » Patch needs review
Status: Needs work » Needs review
FileSize
google_charts_large-2418717-1.patch826 bytes

A patch for this issue was attached to the referenced issue #2418717: Google charts: large number format by geekygnr

In my test the patch worked: the thousands operator is included and localized.

I attach this patch to this issue too, but credit belongs to geekynr.

Fabianx’s picture

Fabianx CreditAttribution: Fabianx as a volunteer and at Tag1 Consulting commented
Title: Patch needs review » Include thousands separator in chart display
mstrelan’s picture

mstrelan CreditAttribution: mstrelan commented

Different approach for Highcharts, something like this (global).

  Highcharts.setOptions({
    lang: {
      decimalPoint: '.',
      thousandsSep: ','
    }
  });
kellyimagined’s picture

kellyimagined CreditAttribution: kellyimagined commented
FileSize
highcharts.js_.txt188.18 KB

Updating the highcharts.js library file to "," instead of " " was the solution I ended up using.

Remove the .txt before uploading.