mdpnp / mdpnp

MD PnP OpenICE
https://mdpnp.mgh.harvard.edu/
113 stars 68 forks source link

Do not animate charts in the WIS demo #105

Open simonkelly1973 opened 4 months ago

simonkelly1973 commented 4 months ago

At the moment when adding data to the charts in the web integration services demo, animation is used and this can show that the new point "drops all the way down to the x-axis" on the y-axis, before "animating" back to the actual point it's supposed to be at. I think this might be a consequence of one of these options

https://www.chartjs.org/docs/latest/configuration/animations.html#easing

but there are certainly other axis specific animations.

Comments were received that this form of animation is confusing in the way it represents the data.

simonkelly1973 commented 4 months ago

059e94f just sets animation to 'none' during update. The longer term thing is probably to look at all those 'easing' options, or possibly to reintroduce a custom canvas. All for some point in the future...