From 0c9f5d982b1ba9cc5b8405d75f7454d30e13b475 Mon Sep 17 00:00:00 2001
From: Ben Gazzard <ben@dartoxia.com>
Date: Tue, 5 Dec 2023 14:09:08 +0000
Subject: [PATCH] Update breadcrumbs per designs

---
 .../web/src/components/KurtosisBreadcrumbs.tsx    | 14 +++++---------
 .../web/src/emui/enclaves/EnclaveRoutes.tsx       | 15 ++++++++++-----
 2 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/enclave-manager/web/src/components/KurtosisBreadcrumbs.tsx b/enclave-manager/web/src/components/KurtosisBreadcrumbs.tsx
index f10b798379..611e3492f7 100644
--- a/enclave-manager/web/src/components/KurtosisBreadcrumbs.tsx
+++ b/enclave-manager/web/src/components/KurtosisBreadcrumbs.tsx
@@ -215,15 +215,11 @@ type KurtosisBreadcrumbItemProps = KurtosisBreadcrumb & {
 };
 
 const KurtosisBreadcrumbItem = ({ name, destination, alternatives, isLastItem }: KurtosisBreadcrumbItemProps) => {
-  if (isLastItem) {
-    return (
-      <Text fontSize={"xs"} fontWeight={"semibold"} p={"2px 8px"} borderRadius={"6px"} bg={"gray.650"}>
-        {name}
-      </Text>
-    );
-  }
-
-  const baseLink = (
+  const baseLink = isLastItem ? (
+    <Text fontSize={"xs"} fontWeight={"semibold"} p={"2px 8px"} borderRadius={"6px"} bg={"gray.650"}>
+      {name}
+    </Text>
+  ) : (
     <BreadcrumbLink as={Link} to={destination}>
       <Button variant={"breadcrumb"} size={"xs"}>
         {name}
diff --git a/enclave-manager/web/src/emui/enclaves/EnclaveRoutes.tsx b/enclave-manager/web/src/emui/enclaves/EnclaveRoutes.tsx
index a9cbcd11af..d1c1f73208 100644
--- a/enclave-manager/web/src/emui/enclaves/EnclaveRoutes.tsx
+++ b/enclave-manager/web/src/emui/enclaves/EnclaveRoutes.tsx
@@ -3,6 +3,7 @@ import { FilesArtifactNameAndUuid, ServiceInfo } from "enclave-manager-sdk/build
 import { FiPlus } from "react-icons/fi";
 import { Outlet, Params } from "react-router-dom";
 import { GoToEnclaveOverviewButton } from "../../components/enclaves/GotToEncalaveOverviewButton";
+import { isDefined } from "../../utils";
 import { RemoveFunctions } from "../../utils/types";
 import { KurtosisEnclavesRouteObject } from "../types";
 import { Artifact } from "./enclave/artifact/Artifact";
@@ -96,13 +97,13 @@ export const enclaveRoutes = (): KurtosisEnclavesRouteObject[] => [
                   crumb: (data: RemoveFunctions<EnclavesState>, params: Params<string>) => {
                     const activeTab = params.activeTab;
 
-                    let routeName = activeTab?.toLowerCase() === "logs" ? "Logs" : "Overview";
+                    if (!isDefined(activeTab) || activeTab.toLowerCase() === "overview") {
+                      return [];
+                    }
 
                     return {
-                      name: routeName,
-                      destination: `/enclave/${params.enclaveUUID}/service/${params.serviceUUID}/${
-                        params.activeTab || "overview"
-                      }`,
+                      name: "Logs",
+                      destination: `/enclave/${params.enclaveUUID}/service/${params.serviceUUID}/logs`,
                     };
                   },
                 },
@@ -170,6 +171,10 @@ export const enclaveRoutes = (): KurtosisEnclavesRouteObject[] => [
               crumb: (data: RemoveFunctions<EnclavesState>, params: Params<string>) => {
                 const activeTab = params.activeTab;
 
+                if (!isDefined(activeTab) || activeTab.toLowerCase() === "overview") {
+                  return [];
+                }
+
                 let routeName =
                   activeTab?.toLowerCase() === "logs"
                     ? "Logs"