Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace Victory Charts with Lightweight Charting Library #263

Open
mikhael28 opened this issue May 31, 2022 · 2 comments
Open

Replace Victory Charts with Lightweight Charting Library #263

mikhael28 opened this issue May 31, 2022 · 2 comments
Labels
React Involves the React framework

Comments

@mikhael28
Copy link
Owner

VictoryCharts takes up a good amount of space in our stats.html breakdown of npm modules that is generated when we run vite build. We are only using one chart, and it's importing a ridiculous amount of node modules for it.

Additionally, when we run pnpm install it routinely complains about Victory Charts not having a required peer of React 16 or whatever. If you have experience with a particular charting library, can you recommend one that has a lighter kb footprint? I have a soft spot for Victory, because Formidable is based in Seattle, but I'm open to change.
@mikhael28 mikhael28 added the React Involves the React framework label May 31, 2022
@jayeclark
Copy link
Collaborator

What about D3 or Britecharts (react wrapper for D3)? It looks like a huge package when importing but IIRC it tree shakes pretty well. And part of Victory seems to be based on it anyway.

@mikhael28
Copy link
Owner Author

D3 might be cool, I've heard good things - it seems like it's its own ecosystem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
React Involves the React framework
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mikhael28 @jayeclark