Jukka Kurkela
Jukka Kurkela
This leads to another related change needed in docs: https://www.chartjs.org/docs/master/getting-started/integration.html Should probably have ES Modules at top, update the CommonJS part and what about the Require JS?
Using esbuild? I think most others can shake assigning class properties (it can have side effects, but usually does not). I'm not against changing to static properties, but its a...
Changing es version can break things.
I think this is one big thing to drive towards v4. I think we need to do 3.8.1 and 3.9 first. There has been some talk on moving to typescript...
We need to merge these in some sane order to avoid tough rebases
Isn't that the same build that would end up in cdn? If tests needs. `treeshake: false`, then the umd build would not be complete, right?
I committed the dist/chart.js to a test repository for a diff. This is what changes on the output with these additions: https://github.com/kurkle/chart-test/commit/df9562b9fcdbae96c7827d7c23a6fc38d395f5ea
Give thumbs up, if this is something you'd like added. I'll only finish this if it gets enough votes.
There is a jump at the end of the animation, when the arcs reach 180 degree circumference. That is another problem. I think a proper fix would be getting rid...
I experimented a little with it. Replaced https://github.com/chartjs/Chart.js/blob/7432b609c487e1b17c8000fe9f94c58fa65936b8/src/elements/element.arc.js#L340-L342 with ```js const fix = 1 + Math.sin(-Math.min(PI, circumference)); radiusOffset = offset / 2 * fix; ``` to make the animations smooth...