fix: When the Rebalance tab is selected, the Brokers tab keeps the ap…

…pearance of being selected as well. (#1125) Signed-off-by: hemahg <[email protected]>
streamshub · Oct 23, 2024 · 762965a · 762965a
1 parent a26c83b
commit 762965a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/ui/app/[locale]/(authorized)/kafka/[kafkaId]/@header/nodes/page.tsx b/ui/app/[locale]/(authorized)/kafka/[kafkaId]/@header/nodes/page.tsx
@@ -70,7 +70,7 @@ function Header({
         <PageNavigation>
           <Nav aria-label="Node navigation" variant="tertiary">
             <NavList>
-              <NavItemLink url={`/kafka/${kafkaId}/nodes`}>
+              <NavItemLink url={`/kafka/${kafkaId}/nodes`} exact={true}>
                 Overview
               </NavItemLink>
               {cruiseControlEnable && (

diff --git a/ui/components/Navigation/NavItemLink.tsx b/ui/components/Navigation/NavItemLink.tsx
@@ -14,8 +14,9 @@ export function NavItemLink<T extends string>({
 }>) {
   const pathname = usePathname();
   const isActive = exact
-    ? url === pathname
-    : pathname.startsWith(url.toString());
+    ? pathname === url.toString()
+    : pathname.startsWith(url.toString()) && pathname === url.toString();
+
   return (
     <NavItem isActive={isActive}>
       <Link href={url}>{children}</Link>