grofit / aurelia-chart

A chart element for aurelia which is powered by chart js using html5 canvas.

MIT License

46 stars 25 forks source link

Entire chart is redrawn when values change #39

Open zshall opened 3 years ago

zshall commented 3 years ago

Steps to Reproduce

Create a pie chart or doughnut chart which uses animation and updates using should-update="true
Change the chart's data (bound to chartData.datasets[0].data)

Expected Results

The chart will update in the way that it does in ChartJS demos, smoothly updating just what has changed: Smooth updates

Actual Results

The chart completely redraws, starting with the rotation of the wedges of the pie chart: Redrawn completely

You can reproduce this bug yourself on the example site

This may be a side effect of how the chart options are passed into ChartJS.

grofit commented 3 years ago

There used to be issues with the refreshes (As this was originally made when it was chartjs 1.x) but I don't honestly know the best way to do it now as I have just left this repo as is, feel free to upload a PR if you find the solution.