haiiaaa / chartjs-gauge

Gauge chart for Chart.js
53 stars 35 forks source link

Please remove the sort from the update override #23

Open MoistCarrot opened 3 years ago

MoistCarrot commented 3 years ago

The GaugeController is currently sorting the dataset that is passed into it via this line of code: https://github.com/haiiaaa/chartjs-gauge/blob/2ea50541d754d710cb30c2502fa690ac5dc27afd/src/controllers/controller.gauge.js#L216

This should be removed, or made configurable. I need my data to be displayed in the order that I pass it in as; not a sorted version of it.

ChartJS doughnuts display data in datasets in sequential order. There isn't really a reason for the gauge chart to sort those values by default. If the user wants their data to be sorted, they can sort the dataset themselves, and pass in that sorted dataset into the update() function.

Please let me know your thoughts on this; thanks!