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 #156

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from
Open

Code splitting #156

wants to merge 14 commits into from

Conversation

NeoLegends
Copy link
Member

@NeoLegends NeoLegends commented Apr 29, 2018

We configure Rollup to build two versions of Festify. One for browsers with native ES module support, the other one using SystemJS.

Primary goal is to remove the Polymer and firebase/database dependencies from the app-shell bundle (respectively the initial bundle), which will greatly reduce the code we have to load upfront. See #147.

@NeoLegends NeoLegends force-pushed the feature/lazy-loading branch from 34a4486 to 84c2944 Compare May 26, 2018 11:37
@mraerino mraerino force-pushed the feature/lazy-loading branch from 55bc10c to b4fd999 Compare May 31, 2018 11:50
This is currently done less optimally than possible but works alright.
The new build also supports browsers with a shitty JS Engine, so there shouldn't be any problems anymore.
@NeoLegends NeoLegends force-pushed the feature/lazy-loading branch from bfd0fbc to e0f1557 Compare June 1, 2018 19:28
@mraerino mraerino force-pushed the develop branch 2 times, most recently from 9f5a930 to 9e40e3f Compare April 23, 2019 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants