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

[8.x] [Kibana Overview] Shift SCSS import and lazy-load main component (#204661) #205484

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…stic#204661)

## Summary

Part of elastic/kibana-team#1082

The purpose of this PR is to reduce the page load bundle size of the
`kibanaOverview` plugin.

Converts the main component of the `kibanaOverview` plugin to be lazy
loaded, and shifts the import of the main SCSS file to be imported from
the lazily-loaded component. This PR does not include any changes that
would be noticeable by end-users. It changes the internals to use a
different technology for styling components.

## References
1. https://emotion.sh/docs/globals
2. https://emotion.sh/docs/best-practices
3.
elastic/eui#6828 (comment)

(cherry picked from commit 1861bbc)
@kibanamachine kibanamachine force-pushed the backport/8.x/pr-204661 branch from 384daf2 to 2423b0e Compare January 3, 2025 12:26
@kibanamachine kibanamachine merged commit 07e09d1 into elastic:8.x Jan 3, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
kibanaOverview 49.9KB 60.2KB +10.3KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kibanaOverview 15.1KB 5.3KB -9.8KB
Unknown metric groups

async chunk count

id before after diff
kibanaOverview 2 3 +1

cc @tsullivan

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

Successfully merging this pull request may close these issues.

3 participants