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

Add @babel/plugin-transform-runtime #282

Closed
wants to merge 1 commit into from
Closed

Add @babel/plugin-transform-runtime #282

wants to merge 1 commit into from

Conversation

vtaits
Copy link

@vtaits vtaits commented Jan 31, 2019

@taion
Copy link
Member

taion commented Jan 31, 2019

do you have numbers on what this does for the bundle size? my impression is that the helpers compress really well, so the net benefit of runtime when we don't need its kind of polyfilling is quite low

@jquense
Copy link
Member

jquense commented Jan 31, 2019

It should be a clearer win with Babel 7 I think since the runtime doesn't polyfill anything by default anymore. But we should check

@vtaits
Copy link
Author

vtaits commented Jan 31, 2019

I have screens before and after injecting this plugin to my project.

Before:
image

After:
image

But bundle contains also polyfills and used libraries.

If every library will import same helpers from @babel/runtime bundle size will more small.

@vtaits
Copy link
Author

vtaits commented Jan 31, 2019

With react-overlays: 280K before, 272K after.

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.

3 participants