-
Notifications
You must be signed in to change notification settings - Fork 191
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ChartJS should be updated to 3.x to improve speed and bundle size #5809
Comments
Important Note: |
@JasonTheAdams This is close enough, right? @DevinWalker won't notice a difference 😅
|
There is a lot to dig through, so I don't think I'll be able to complete this upgrade before the next cycle. That said, here are my notes on what I was able to upgrade:
Also, I wasn't able to get the |
This issue is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 14 additional days. |
This issue was closed because it has been stalled for an additional 14 days with no activity. |
Details
We use ChartJS for our reporting and are using ChartJS 2.x. It does a good job, but is absolutely massive and bloats the bundle size (~450kb). This also slows things down. About a month ago ChartJS 3.x came out which supports Tree Shaking. This is really the reason it was so huge.
Updating to ChartJS 3.x will help reduce the bundle size and overall speed.
Additional Context
Migration Guide: https://www.chartjs.org/docs/latest/getting-started/v3-migration.html
Acceptance Criteria
The text was updated successfully, but these errors were encountered: