From a6e6e10c69fbb1ea660ff43ccac677a37741153d Mon Sep 17 00:00:00 2001
From: Ardeman <are.theman@gmail.com>
Date: Wed, 5 Mar 2025 22:39:00 +0800
Subject: [PATCH] fix: simplify path extraction logic in Sidebar component

---
 app/layouts/admin/sidebar.tsx | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/app/layouts/admin/sidebar.tsx b/app/layouts/admin/sidebar.tsx
index 17fd1c2..4d7c0e2 100644
--- a/app/layouts/admin/sidebar.tsx
+++ b/app/layouts/admin/sidebar.tsx
@@ -5,11 +5,8 @@ import { MENU } from './menu'
 
 export const Sidebar = () => {
   const { pathname } = useLocation()
-  const path = () => {
-    // Extract the first three segments for deep paths, otherwise use the full pathname
-    const segments = pathname.split('/')
-    return segments.length > 3 ? segments.slice(0, 3).join('/') : pathname
-  }
+  const segments = pathname.split('/')
+  const path = segments.length > 3 ? segments.slice(0, 3).join('/') : pathname
 
   return (
     <div className="flex min-h-[calc(100dvh-80px)] flex-col gap-y-10 overflow-y-auto bg-white p-5">
@@ -24,19 +21,19 @@ export const Sidebar = () => {
               to={url}
               key={`${group}-${title}`}
               className={twMerge(
-                path() === url ? 'bg-[#707FDD]/10 font-bold' : '',
+                path === url ? 'bg-[#707FDD]/10 font-bold' : '',
                 'group/menu flex h-[42px] w-[200px] items-center gap-x-3 rounded-md px-5 transition-all hover:bg-[#707FDD]/10',
               )}
             >
               <Icon
                 className={twMerge(
-                  path() === url ? 'text-[#5363AB]' : 'text-[#A6ABC8]',
+                  path === url ? 'text-[#5363AB]' : 'text-[#A6ABC8]',
                   'h-[18px] w-[18px] transition-all group-hover/menu:text-[#5363AB]',
                 )}
               />
               <span
                 className={twMerge(
-                  path() === url ? 'text-[#5363AB]' : 'text-[#273240]',
+                  path === url ? 'text-[#5363AB]' : 'text-[#273240]',
                   'text-base transition-all group-hover/menu:text-[#5363AB]',
                 )}
               >