diff --git a/ui/litellm-dashboard/e2e_tests/tests/migration/migratedPages.spec.ts b/ui/litellm-dashboard/e2e_tests/tests/migration/migratedPages.spec.ts index 0b01ca5d280..61e89a95462 100644 --- a/ui/litellm-dashboard/e2e_tests/tests/migration/migratedPages.spec.ts +++ b/ui/litellm-dashboard/e2e_tests/tests/migration/migratedPages.spec.ts @@ -31,9 +31,9 @@ async function expectRendered(page: Page) { async function clickSidebar(page: Page, segment: string) { const link = page.locator(`a[href$="/ui/${segment}"]`).first(); for (let i = 0; i < 8 && !(await link.isVisible().catch(() => false)); i++) { - const closedCollapsible = page.locator('[data-state="closed"] > button[role="menuitem"]').first(); - if (await closedCollapsible.isVisible().catch(() => false)) { - await closedCollapsible.click(); + const closedTrigger = page.locator('button[role="menuitem"]:not([data-panel-open])').first(); + if (await closedTrigger.isVisible().catch(() => false)) { + await closedTrigger.click(); await page.waitForTimeout(250); continue; } diff --git a/ui/litellm-dashboard/src/components/leftnav.tsx b/ui/litellm-dashboard/src/components/leftnav.tsx index 3e0e05565a2..c34f99fa0d9 100644 --- a/ui/litellm-dashboard/src/components/leftnav.tsx +++ b/ui/litellm-dashboard/src/components/leftnav.tsx @@ -453,7 +453,7 @@ function NavItem({ const tooltipLabel = typeof item.label === "string" ? item.label : item.page; return ( - {content} + {tooltipLabel} ); @@ -480,15 +480,17 @@ function CollapsibleNavItem({ const tooltipLabel = typeof item.label === "string" ? item.label : item.page; return ( - - - + setOpen((o) => !o)} + className="flex items-center justify-center h-8 py-1.5 px-2 rounded-md text-[13px] text-sidebar-foreground/70 hover:bg-sidebar-accent transition-colors w-full" + > + {item.icon} + + } + /> {tooltipLabel} ); @@ -496,19 +498,16 @@ function CollapsibleNavItem({ return ( - - + + {item.icon} + {item.label} +