[Serverless/Observability] Get started breadcrumbs need polish #170030
Labels
Project:Serverless
Work as part of the Serverless project for its initial release
Team:APM
All issues that need APM UI Team support
similar to #170029
Missing deeper context breadcrumbs
In serverless breadcrumbs work a bit different from how they work on-prem: they are automatically built from the serverless navigation tree. Additional work needs to be done for serverless to set the deeper context breadcrumbs. Currently get started apps don't set deeper context breadcrumbs
example:
There are multiple ways to add deeper context breadcrumbs, the first one is manual and involves using a new
serverless.setBreadcrumbs
. The apps only need to set the end of breadcrumbs path which isn't already part of the side nav navigation tree.Another way is simpler and it makes the navigation automatically merge the serverless navigational breadcrumbs from the navigation tree with the full breadcrumbs set using old
core.chrome.setBreadcrumbs
api. To opt-in into this apps need to adddeepLinkId
to the root breadcrumbs set incore.chrome.setBreadcrumbs
. The serverless navigation will use thisdeepLinkId
to merge the breadcrumbs set incore.chrome.setBreadcrumbs
with the navigation tree breadcrumbs that are coming from the navigation tree.Here are some examples where we fixed it for some apps:
In some cases it could make sense to fix this on the navigation tree level:
Completely missing breadcrumbs for some pages
Another issue with breadcrumbs is that if a page is accessible from the app, but is not the part of the navigation tree, then the active path can be lost an nothing is highlighted in the side nav and breadcrumbs are empty.
Example:
There are different ways to fix this depending on the use-case:
isActive
check for some of the existing nodes in the navigation tree, to force the navigation path to be considered activeExample:
kibana/x-pack/plugins/serverless_observability/public/components/side_navigation/index.tsx
Line 160 in 9a8277e
sideNavStatus: 'hidden'
this will fix the active path and the breadcrumbskibana/x-pack/plugins/serverless_observability/public/components/side_navigation/index.tsx
Line 43 in 9a8277e
renderAs: 'item',
on the parentThe text was updated successfully, but these errors were encountered: