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

perf(www): low-hanging fruit for reducing gatsbyjs.org bundle #21470

Closed
tesseralis opened this issue Feb 14, 2020 · 3 comments · Fixed by #23209
Closed

perf(www): low-hanging fruit for reducing gatsbyjs.org bundle #21470

tesseralis opened this issue Feb 14, 2020 · 3 comments · Fixed by #23209
Labels
help wanted Issue with a clear description that the community can help with.

Comments

@tesseralis
Copy link
Contributor

tesseralis commented Feb 14, 2020

Recently @pieh took a bundle analyzer off a recent build of gatsbyjs.org:

Gatsby sitemap

I noticed a couple of things that stood out as unnecessary that we should probably fix:

  • An SVG file, 8-double-work.svg used in a recent blog post, is included for some reason
  • A lot of components used in MDX were made global (Make website components used in MDX files globally available #19036) to solve a localization related issue. Some of these components have significant footprints, like layer-model, which pulls in layer-icon. We need to figure out another solution so that these components don't clutter up the bundle.
  • A lot of the algolia-styles can probably be lazy-loaded. Even though the search form is included in every page, a lot of these styles involve the actual dropdown and can be loaded later.
  • FragmentsQuery is included in the bundle for some reason, even though it's only used in the GraphQL API page.
@tesseralis tesseralis added bot: merge on green Gatsbot will merge these PRs automatically when all tests passes help wanted Issue with a clear description that the community can help with. and removed bot: merge on green Gatsbot will merge these PRs automatically when all tests passes labels Feb 14, 2020
@wardpeet
Copy link
Contributor

Thanks for letting us know, i'll be changing our bundle strategy so it might solve this problem partially. .org is a mess though 😛

@github-actions
Copy link

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

@github-actions github-actions bot added the stale? Issue that may be closed soon due to the original author not responding any more. label Mar 19, 2020
@wardpeet wardpeet removed the stale? Issue that may be closed soon due to the original author not responding any more. label Mar 26, 2020
@github-actions
Copy link

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

@github-actions github-actions bot added the stale? Issue that may be closed soon due to the original author not responding any more. label Apr 15, 2020
@tesseralis tesseralis added not stale and removed stale? Issue that may be closed soon due to the original author not responding any more. labels Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Issue with a clear description that the community can help with.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants