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.

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

Work on reducing library size #9419

Closed
avalanche1 opened this issue Feb 26, 2018 · 4 comments
Closed

Work on reducing library size #9419

avalanche1 opened this issue Feb 26, 2018 · 4 comments

Comments

@avalanche1
Copy link

What problem does this feature solve?

The size of library source is WAY too big!
Importing a button increases app size by 95kb! A single button, omg! 28kb is the Button/index.css & 67kb is antd/index.css
Importing single Datepicker component increases my app size for 354kb!! (ungzipped):

  • 71kb - antd/style/index.css
  • 28kb - Button/style/index.css
  • 43kb - date-picker/style/index.css
  • 30kb - date-picker/misc js files
  • 9kb - time-picker/style/index.css
  • 10kb - input/style/index.css
  • 53kb - rc-calendar minified
  • 52kb - moment
  • 20kb - rc-time-picker
  • 18kb - rc-trigger
  • 14kb - dom-align
  • 9kb - rc-animate
  • 6kb - rc-util

Something has to be done about that - having single component weigh in additional 354kb - that's not possible in 2018.

What does the proposed API look like?

no api changes

@yesmeck
Copy link
Member

yesmeck commented Feb 26, 2018

@avalanche1 Have you tried to analyze your bundle by webpack visualizer?

@yesmeck
Copy link
Member

yesmeck commented Feb 26, 2018

We keep eyes on library size, If you have a more specific suggestion can reduce the library size, welcome leave a comment here. I'm going to close this issue because no actions we can take for now.

@yesmeck yesmeck closed this as completed Feb 26, 2018
@skizzo
Copy link

skizzo commented Jul 8, 2018

Did anybody come up with something that works? My bundle size is huge and contains stuff i definitely don't need (or implement) anywhere.. Thanks!

@ghost
Copy link

ghost commented Mar 11, 2019

@skizzo @avalanche1

Maybe this is what you need.
iamkun/dayjs#529

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants