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

refactor(client): load more lazily #10143

Closed
wants to merge 4 commits into from
Closed

refactor(client): load more lazily #10143

wants to merge 4 commits into from

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Dec 6, 2023

Summary

Problem

Our main bundle is larger than it could be.

Solution

Load more pages lazily.


Screenshots

Before

  156.44 kB  build/static/js/main.1b11e75b.js
  28.42 kB   build/static/css/main.8782317e.css

After

  128.77 kB  build/static/js/main.cfa13c20.js
  21.99 kB   build/static/css/main.a25ea272.css

How did you test this change?

Ran yarn build:client.

@caugner caugner requested a review from a team as a code owner December 6, 2023 18:43
@github-actions github-actions bot added plus work around features related to MDN Plus blog labels Dec 6, 2023
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 Please rebase onto or merge the latest main. label Dec 14, 2023
Reduces the main bundle significantly:
- main.js: 156.44 kB -> 128.77 kB (- 18 %)
- main.css: 28.42 kB -> 21.99 kB (- 17 %)
@github-actions github-actions bot removed the merge conflicts 🚧 Please rebase onto or merge the latest main. label Dec 14, 2023
@caugner caugner marked this pull request as draft January 2, 2024 10:45
@github-actions github-actions bot added the idle label Feb 1, 2024
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added merge conflicts 🚧 Please rebase onto or merge the latest main. and removed idle labels Feb 21, 2024
@github-actions github-actions bot added the idle label Mar 24, 2024
@caugner
Copy link
Contributor Author

caugner commented May 2, 2024

Lazy-loading impacts SSR, so closing for now, see also: #11048

@caugner caugner closed this May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog idle merge conflicts 🚧 Please rebase onto or merge the latest main. plus work around features related to MDN Plus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant