Skip to content

[7.x] [App Search] Convert Schema pages to new page template (#102846)#102996

Merged
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-102846
Jun 22, 2021
Merged

[7.x] [App Search] Convert Schema pages to new page template (#102846)#102996
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-102846

Conversation

@kibanamachine
Copy link
Contributor

Backports the following commits to 7.x:

* Convert Schema page to new page template

+ update empty state - remove panel wrapper, add create schema field modal

* Convert ReindexJob view to new page template

+ remove breadcrumb prop

* Convert Meta Engine Schema view to new page template

* Update routers

* [Polish] Misc Davey Schema UI tweaks

- see https://github.com/elastic/kibana/pull/101958/files

+ change color away from secondary, since that's going away in EUI at some point

* [UX] Fix SchemaAddFieldModal stuttering on first new schema field add

- With the new template, transitioning from the empty state to the filled schema state causes the modal to stutter due to the component rerender

- Changing the page to not instantly react/update `hasSchema` when local schema state changes but instead to wait for the server call to finish and for cachedSchema to update fixes the UX problem

* [UI polish] Revert button color change per Davey's feedback
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 22, 2021
@kibanamachine kibanamachine enabled auto-merge (squash) June 22, 2021 19:29
@kibanamachine
Copy link
Contributor Author

💚 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 2.1MB 2.1MB -826.0B

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

cc @constancecchen

@kibanamachine kibanamachine merged commit 089a175 into elastic:7.x Jun 22, 2021
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