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

Use deep import for inline-style-prefixer #363

Merged
merged 1 commit into from
Feb 19, 2019
Merged

Conversation

lencioni
Copy link
Collaborator

After updating to 2.3.0, I noticed a large bundle size increase. I
believe this is due to switching this from a deep import to a named
import, so we were duplicating a bunch of code (the named import used
the es build, which brought in the es plugins, but our generated data
brought in the lib plugins).

As a quick fix, I think we can bring bundle sizes back down by using a
deep import here again.

After updating to 2.3.0, I noticed a large bundle size increase. I
believe this is due to switching this from a deep import to a named
import, so we were duplicating a bunch of code (the named import used
the es build, which brought in the es plugins, but our generated data
brought in the lib plugins).

As a quick fix, I think we can bring bundle sizes back down by using a
deep import here again.
@lencioni lencioni requested a review from cyejia February 19, 2019 18:44
@lencioni lencioni merged commit f3cca9b into master Feb 19, 2019
@lencioni lencioni deleted the prefixer-deep-import branch February 19, 2019 18:49
@lencioni
Copy link
Collaborator Author

Bundlephobia is much happier with this. https://bundlephobia.com/result?p=aphrodite@2.3.1

image

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

Successfully merging this pull request may close these issues.

None yet

2 participants