4 ms·
That is an UMD bundle. What is more disappointing is that Chartjs itself seems to be written in ES5.
by fleetfox 7y ago
That is an UMD bundle. What is more disappointing is that Chartjs itself seems to be written in ES5.
- ergo14 7y agoWhy it still works, no need to refactor the code for the sake of refactoring if this is free work. Change to ES modules would be welcome though.
- vimslayer 7y agoThey seem to be using Rollup for builds, which supports ESM output, so they wouldn't even need to do any refactoring for that. Couple of configuration changes would suffice.
- benmccann 7y agoI'd be happy to review any PR to improve our rollup & package.json config as long as it's backwards compatible with the documented methods of using Chart.js: https://www.chartjs.org/docs/latest/getting-started/integration.html https://www.chartjs.org/docs/latest/getting-started/integrat...