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
Fix lodash ES module imports #2
Comments
Closed
philippfromme
added a commit
that referenced
this issue
Aug 1, 2018
ghost
added
in progress
Currently worked on
and removed
ready
Ready to be worked on
labels
Aug 1, 2018
philippfromme
added
needs review
Review pending
and removed
in progress
Currently worked on
labels
Aug 1, 2018
ghost
assigned philippfromme
Aug 1, 2018
nikku
pushed a commit
that referenced
this issue
Aug 1, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bundling the library with browserify / babelify yields the following output:
It hints us on the fact that the full distribution of lodash (> 500KB) is being packed into the properties panel.
To properly consume lodash so that ES module aware bundlers can tree-shake it we must not use destructuring syntax:
Instead, we must import individual utilities like this:
The alternative is to use lodash-es. However, that library variant has tree-shaking issues, too (as far as I am aware).
The text was updated successfully, but these errors were encountered: