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

Replace lodash with lodash.<fn> deps #41

Closed
Closed
Replace lodash with lodash.<fn> deps#41
@jakubzitny

Description

@jakubzitny
jakubzitny
opened on Aug 23, 2016 · edited by jakubzitny

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?

Activity

jakubzitny
mentioned this on Aug 23, 2016
billyjanitsch

billyjanitsch commented on Sep 1, 2016

@billyjanitsch
billyjanitsch
on Sep 1, 2016
Collaborator

See react-dnd/react-dnd#517 (comment).

jakubzitny

jakubzitny commented on Sep 3, 2016

@jakubzitny
jakubzitny
on Sep 3, 2016
Author

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

jakubzitny
closed this as completedon Sep 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @billyjanitsch@jakubzitny

        Issue actions
          Replace lodash with lodash.<fn> deps · Issue #41 · react-dnd/react-dnd-html5-backend