Skip to content
This repository was archived by the owner on Jun 18, 2018. It is now read-only.

Replace lodash with lodash.<fn> deps #41

Closed
jakubzitny opened this issue Aug 23, 2016 · 2 comments
Closed

Replace lodash with lodash.<fn> deps #41

jakubzitny opened this issue Aug 23, 2016 · 2 comments

Comments

@jakubzitny
Copy link

jakubzitny commented Aug 23, 2016

Hey there, we're using react-dnd-html5-backend from NPM and we're quite surprised by the size of the library. The whole react-dnd is taking 4.7M and almost all of that is only lodash. From lodash you're using only memoize, union, without and defaults functions. These are available separately from NPM taking only 148K.

I've already reported similar issue in react-dnd and also #39 might be related to this.

Would you be interested in a PR?

@billyjanitsch
Copy link
Collaborator

@jakubzitny
Copy link
Author

Closing based on the conversation at react-dnd/react-dnd#517.

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

No branches or pull requests

2 participants