From ae5aabb036f9d2b43e6fc2abe7f113273050f057 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Thu, 29 Feb 2024 11:58:59 -0500
Subject: [PATCH] Refactoring Breadcrumb component - refs BT#20933 BT#20950

---
 assets/css/scss/layout/_breadcrumb.scss |  2 +-
 assets/vue/components/Breadcrumb.vue    | 43 +++++++++++--------------
 2 files changed, 19 insertions(+), 26 deletions(-)

diff --git a/assets/css/scss/layout/_breadcrumb.scss b/assets/css/scss/layout/_breadcrumb.scss
index c6c40cb15d..0826a28e79 100644
--- a/assets/css/scss/layout/_breadcrumb.scss
+++ b/assets/css/scss/layout/_breadcrumb.scss
@@ -1,5 +1,5 @@
 .app-breadcrumb {
-  @apply bg-white pb-3 mb-2 text-tiny font-semibold leading-4;
+  @apply bg-white pb-3 text-tiny font-semibold leading-4 space-y-1;
 
   ol {
     @apply mb-1 flex gap-1 text-gray-90;
diff --git a/assets/vue/components/Breadcrumb.vue b/assets/vue/components/Breadcrumb.vue
index 71ddfe42a3..df4a828305 100644
--- a/assets/vue/components/Breadcrumb.vue
+++ b/assets/vue/components/Breadcrumb.vue
@@ -1,8 +1,10 @@
 <template>
-  <div class="app-breadcrumb">
+  <div
+    v-show="itemList.length > 0"
+    class="app-breadcrumb"
+  >
     <Breadcrumb
-      :home="home"
-      :model="foo"
+      :model="itemList"
       unstyled
     >
       <template #item="{ item, props }">
@@ -69,14 +71,8 @@ const { t } = useI18n()
 
 const { course, session } = storeToRefs(cidReqStore)
 
-const home = {
-  icon: "mdi mdi-home-variant",
-  route: { name: "Home" },
-}
-
-const foo = computed(() => {
+const itemList = computed(() => {
   const list = [
-    "CourseHome",
     "MyCourses",
     "MySessions",
     "MySessionsUpcoming",
@@ -109,6 +105,18 @@ const foo = computed(() => {
     return items
   }
 
+  if (course.value) {
+    if (session.value) {
+      items.push({
+        label: t("My sessions"),
+      })
+    } else {
+      items.push({
+        label: t("My courses"),
+      })
+    }
+  }
+
   if (componentProps.legacy.length > 0) {
     const mainUrl = window.location.href
     const mainPath = mainUrl.indexOf("main/")
@@ -137,21 +145,6 @@ const foo = computed(() => {
         route: { name: "CourseHome", params: { id: course.value.id }, query: route.query },
       })
     }
-
-    const { path, matched } = route
-    const lastItem = matched[matched.length - 1]
-
-    matched.forEach((pathItem) => {
-      if (pathItem.path) {
-        const isDisabled = route.path === path || lastItem.path === route.path
-
-        items.push({
-          label: pathItem.name,
-          disabled: isDisabled,
-          url: isDisabled ? undefined : pathItem.path,
-        })
-      }
-    })
   }
 
   return items