[chrome] Improve breadcrumb extension#209765
Conversation
|
Pinging @elastic/appex-sharedux (Team:SharedUX) |
…prove-breadcrumb-extension
tsullivan
left a comment
There was a problem hiding this comment.
LGTM. It's a slew of improvements!
…prove-breadcrumb-extension # Conflicts: # x-pack/solutions/observability/plugins/observability_logs_explorer/public/components/logs_explorer_top_nav_menu.tsx
jennypavlova
left a comment
There was a problem hiding this comment.
Obs ux infra and services changes LGTM
|
Starting backport for target branches: 8.x https://github.com/elastic/kibana/actions/runs/13202204703 |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
Page load bundle
History
|
💔 All backports failed
Manual backportTo create the backport manually run: Questions ?Please refer to the Backport tool documentation |
|
Friendly reminder: Looks like this PR hasn’t been backported yet. |
Summary
fix #208728
This PR improves breadcrumb extension point for adding starring next to a dashboard breadcrumb #200315:
To test:
data.search.sessions.enabled: trueand see that search session UI appears in solution nav.chrome.setBreadcrumbsAppendExtension, e.g. insrc/platform/plugins/shared/data/public/search/search_service.ts. This actually gonna be used in [Dashboards] Favorite a dashboard from within #200315