-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Fleet] Lazy load package icons in integrations grid #132455
Conversation
Pinging @elastic/fleet (Team:Fleet) |
Looks like we did not sync correctly on this one, #132450 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 I tried to do some profiling in a dev environment but I did not get obvious result this is a real improvement (But it probably still nice to have)
(cherry picked from commit 6cf8ebf)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
(cherry picked from commit 6cf8ebf) # Conflicts: # x-pack/plugins/fleet/public/components/package_icon.tsx
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
…#133165) * [Fleet] Lazy load package icons in integrations grid (#132455) (cherry picked from commit 6cf8ebf) # Conflicts: # x-pack/plugins/fleet/public/components/package_icon.tsx * Fix linting Co-authored-by: Jen Huang <its.jenetic@gmail.com> Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Resolve #122660
Will prevent the package icons from loading until they're near the viewport when scrolling. Reduces request backlog/queueing on HTTP/1.1 connections (which is only what Kibana supports).