frappe / charts

Simple, responsive, modern SVG Charts with zero dependencies
https://frappe.io/charts
MIT License
14.9k stars 716 forks source link

Legend on aggregate charts is not updated on full update #329

Closed helge79 closed 3 years ago

helge79 commented 3 years ago

Expected Behaviour

When using a full data update via chart.update() on an aggregate chart both chart and legend below update

Actual Behaviour

The legend is not updated. If you resize the window (or switch phone's orientation) the legend will get updated with the currently displayed values

Steps to Reproduce:

Frappé Charts version: 1.5.7 JSFiddle: https://jsfiddle.net/z2ckgrtm/

helge79 commented 3 years ago

I added a PR. Simply updating the legend as I do solves the issue but without an animation it looks of course a bit jerky