-
Notifications
You must be signed in to change notification settings - Fork 26
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
huge filesize overhead - twice the size of react+redux implementation? #26
Comments
Looks like it's because we're inlining PropTypes and possibly another dependency (which seems like an error!) |
awesome, glad to hear it's just a bug! Curious if you had any headway with this issue @developit? I'm sure you're quite busy so I'd be happy to help if you can point me in the right direction. :) |
I spent a good long time looking into this yesterday, and it's a result of changes in |
thank you for the update & the investigation! 3.7kb is definitely a notable improvement. Just out of curiosity what were the changes made upstream to lead to this? are they inlining proptypes like you suspected? Cheers! |
I am also interested in learning what changes were made to get the final size down to 3.7kb. |
Hiya, sorry for dropping the ball on this. I'll try to dig up the branch I was working on. |
hi!
we're considering using preact + preact-redux as an alternative to react + react-redux. This is pretty much solely due to bundle size savings migrating to preact.
However preact-redux seems to take up a crazy amount of space. On a fresh preact-cli app with only 1 additional dependency outside of redux and preact-redux, preact-redux is 35% of the total bundle size at 41.8KB on a production build. Way larger than preact, preact-router, polyfills, etc.
Meanwhile in a react app production build I am seeing redux @ 5.76KB + react-redux @ 11.79KB === 17.55KB total, versus preact-redux's 41.9KB. Something doesn't seem right here!
Both are using latest versions of respective libraries (Redux + react/preact-redux).
any ideas how we can bring this bundle size down? It's unclear to me why this would be over twice the filesize.
The text was updated successfully, but these errors were encountered: