search

scottopolis / edd-metrics

EDD Metrics
46 stars 12 forks source link

Add currency formatting for chart tooltips #24

Open pderksen opened 7 years ago

pderksen commented 7 years ago

Probably after #14

scottopolis commented 7 years ago

Can you give me more detail here?

pderksen commented 7 years ago

Sorry for the vagueness. I was referring to the tooltips in the "Earnings by" pie charts. The figures here should be wrapped with appropriate currency formatting such as $1,234.56.

For example, this tooltip number should show up as $794.60.

image

scottopolis commented 7 years ago

Gotcha, will do

Netzberufler commented 7 years ago

ChartJS needs unformatted number values to display the pie charts properly, so it is not possible to change the currency formatting in PHP (I tried that). I guess this needs to be done in JavaScript with a custom Tooltip Template for the ChartJS plugin.