Skip to content

[7.x] [Enterprise Search] Share Loading component (#83246)#83366

Merged
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-83246
Nov 13, 2020
Merged

[7.x] [Enterprise Search] Share Loading component (#83246)#83366
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-83246

Conversation

@cee-chen
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* Add basic Loading component

- for pages with shifting layouts or where skeletal loading doesn't make sense

* Move Loading to shared components

* Update WS to use new shared Loading component

* Fix for non-Layout WS Overview page

* Update x-pack/plugins/enterprise_search/public/applications/app_search/components/engine/engine_router.tsx

Co-authored-by: Scotty Bollinger <scotty.bollinger@elastic.co>

Co-authored-by: Scotty Bollinger <scotty.bollinger@elastic.co>
@cee-chen cee-chen added the backport This PR is a backport of another PR label Nov 13, 2020
@kibanamachine
Copy link
Copy Markdown
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
enterpriseSearch 698.4KB 699.5KB +1.1KB

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@cee-chen cee-chen merged commit 20aa1c4 into elastic:7.x Nov 13, 2020
@cee-chen cee-chen deleted the backport/7.x/pr-83246 branch November 13, 2020 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants