feat(slo): store search state in url

[kdelemme]

Resolves #167899

🍒 Summary

This PR synchronizes the search bar with the url state, useful for bookmarking and sharing search filters.
I've also refactored the search bar to remove the possibility to unselect a sort option.

search.state.mov

[ghost]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

• /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
• /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
• run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

[elasticmachine]

Pinging @elastic/actionable-observability (Team: Actionable Observability)

[kibana-ci]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: ca312fc
• Storybooks Preview

Failed CI Steps

• Osquery Cypress Tests #2
• Serverless Osquery Cypress Tests #4

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
observability	481	482	+1

Async chunks

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

id	before	after	diff
observability	1.0MB	1.0MB	+551.0B

History

• 💔 Build #167823 failed 044a21f
• 💚 Build #166782 succeeded 2a12917
• 💔 Build #166758 failed c56e29d

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

cc @kdelemme

[CoenWarmer]

Looks good. Have you considered placing the logic for use_url_search_state inside a locator? That way other pages inside O11y and other apps could also deeplink to it.

[kdelemme]

Looks good. Have you considered placing the logic for use_url_search_state inside a locator? That way other pages inside O11y and other apps could also deeplink to it.

I'll see how this works

[simianhacker]

LGTM

[mgiota]

Have you considered placing the logic for use_url_search_state inside a locator?

@CoenWarmer Are locators similar to state containers? Cause I have implemented something similar with state containers in the o11y rules page.

In case it's relevant @kdelemme you can have a look at these PRs to checkout how I used state containers
#131523
#131300

My initial inspiration was this PR

[CoenWarmer]

IMO the state containers pattern is a little complex for what it aims to achieve.

I've created a couple of locators which are a bit more simple here:

• x-pack/plugins/observability/public/locators

[kdelemme]

I've created an issue for adding the locator: #169101