From 13c65c75d8ee4631aa450abea78f4f49c0883c8c Mon Sep 17 00:00:00 2001 From: Kawika Avilla Date: Wed, 12 Jul 2023 10:55:43 -0700 Subject: [PATCH] Update breadcrumb title of isDashboardNavMenu (#142) https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4453 Modified the breadcrumbs from the dashboard plugin, to be consistent with the actual name of the application. We should consider not hard checking the name of the breadcrumb but really hooking into the navigation plugin that provides the ability to do what this logic is doing without being hardcoded. https://github.com/opensearch-project/dashboards-reporting/issues/118 Signed-off-by: Kawika Avilla (cherry picked from commit 577871d1ded7d478622964ea23a5f61ed4b0f2e2) --- public/components/context_menu/context_menu.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/public/components/context_menu/context_menu.js b/public/components/context_menu/context_menu.js index 0b7485ab..95c50d4d 100644 --- a/public/components/context_menu/context_menu.js +++ b/public/components/context_menu/context_menu.js @@ -264,7 +264,7 @@ const isDiscoverNavMenu = (navMenu) => { const isDashboardNavMenu = (navMenu) => { return ( (navMenu[0].children.length === 4 || navMenu[0].children.length === 6) && - $('[data-test-subj="breadcrumb first"]').prop('title') === 'Dashboard' + $('[data-test-subj="breadcrumb first"]').prop('title') === 'Dashboards' ); };