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

Code Splitting based on functionality used #277

Closed
andreapernici opened this issue Aug 30, 2021 · 7 comments
Closed

Code Splitting based on functionality used #277

andreapernici opened this issue Aug 30, 2021 · 7 comments
Labels

Comments

@andreapernici
Copy link

Motivation

Considering the growing importance of speed and the upcoming Core Web Vitals metric usage in Google could be a good idea to let developers include only needed features in the js bundle reducing a lot the weight of it in most of the cases.

Design Alternatives

Code splitting.


Something similar to this https://twitter.com/Firebase/status/1430586974318989315

@HarelM
Copy link
Collaborator

HarelM commented Aug 30, 2021

We are in the final stages of migrating to typescript so this might be better handled going forward.
Having said that, I'm not sure how to push such an effort.
If you would like to push this forward feel free. I world suggest to start with a design document to present the required changes.
Good luck :-)

@StephenStrickland
Copy link

@HarelM is there a ETA on typescript, maybe a site/branch where we can monitor progress? @andreapernici are you interested in heading this effort up?

@wipfli
Copy link
Contributor

wipfli commented Sep 18, 2021

We merged the typescript pull request into main. Pre-releases of version 2 are on NPM. Best place to monitor status is probably the changelog.

@wipfli
Copy link
Contributor

wipfli commented Nov 15, 2021

This seems related to named exports. See the discussion in #623.

@HarelM
Copy link
Collaborator

HarelM commented Nov 15, 2021

I'm not sure named exports will solve this.
It requires a more granular approach to architecture and design - exporting a map object that does everything is exactly the opposite for example.
Unfortunately, I don't think we know the code good enough to support this at the moment.
This is somewhat similar to this request #166 - i.e. be able to use just part of this library or part of the funcionality.
We should aim to get there, but I don't think we are right now... :-(

@github-actions
Copy link
Contributor

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label May 15, 2022
@github-actions
Copy link
Contributor

This issue was closed because it has been stalled for 30 days with no activity.

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

No branches or pull requests

4 participants