-
Notifications
You must be signed in to change notification settings - Fork 467
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
[Question]: Installation / peer deps #1037
Comments
I would like to be assigned for this! |
I've double checked the installation guide, a couple of observations here:
@tailwind base;
@tailwind components;
@tailwind utilities; in case they didn't follow tailwindcss installation guide. I'm still trying to figure out how to minimize the bundle size. Now the bottleneck is to find a reliable and reproducible bundle checker. Take bundlephobia as an example, while we get the same result as you, we get a different breakdown when only uploading While it gives similar total size, the For discussion reference, the current installation guide for NextJS as June 4, 2024 is: |
@jzfrank Concerning the bundle size, it looks like it might be due to how the recharts library was designed (according to this comment: recharts/recharts#3697 (comment)). Here is the gist of it from Coltin Kifer (one of the library maintainers):
It looks like there is a babel plugin to allow importing only specific modules but unfortunately it is unmaintained (https://github.com/recharts/babel-plugin-recharts). It's surprising that it's not maintained considering the number of people complaining about recharts bundle size. In any case, as far as I understand, unless you are willing to contribute to that babel plugin and that the maintainer is going to merge your contributions, there isn't much to be done about recharts at the moment. 😞 |
I'm reading your installation guide and am confused as to why there are so many things to install that aren't listed among the other peer dependencies on package json.
Specifically these:
The documentation doesn't currently explain when/why these are required and I don't understand why they aren't simply bundled.
Somewhat related: Bundlephobia is showing me a confusing breakdown of the bundle here:
The text was updated successfully, but these errors were encountered: