From 68cb6390ca1f2e28dca0d7958f1af29a72da0919 Mon Sep 17 00:00:00 2001
From: Damola18 <olutokeezekiel18@gmail.com>
Date: Wed, 15 Jan 2025 14:33:14 +0100
Subject: [PATCH] Refactor SidebarItem to handle external links properly

---
 src/components/dashboard/sidebar-item.tsx | 79 ++++++++++++++---------
 1 file changed, 48 insertions(+), 31 deletions(-)

diff --git a/src/components/dashboard/sidebar-item.tsx b/src/components/dashboard/sidebar-item.tsx
index 27790cc9..0daf9963 100644
--- a/src/components/dashboard/sidebar-item.tsx
+++ b/src/components/dashboard/sidebar-item.tsx
@@ -4,6 +4,7 @@ import { Flex, FlexProps, Icon, LinkBox, Image } from '@chakra-ui/react'
 import { dataAttr } from '@chakra-ui/utils'
 import { usePathname } from 'next/navigation'
 import React from 'react'
+import Link from 'next/link'
 import LinkOverlay from '../elements/LinkElements/LinkOverlay'
 import { useTranslations } from 'next-intl'
 
@@ -20,41 +21,57 @@ export const SidebarItem = (props: SidebarItemProps) => {
 
   const pathname = usePathname()
   const isActiveRoute = pathname?.endsWith(item.route)
+  const isExternalLink = item.route.startsWith('http')
+
+  const renderLinkContent = () => (
+    <Flex
+      h="40px"
+      align="center"
+      pl={{ base: 5, lg: '15' }}
+      cursor="pointer"
+      data-active={dataAttr(isActiveRoute)}
+      color="grayText"
+      fontWeight="medium"
+      _hover={{
+        bg: 'divider',
+        color: 'dimmedText',
+      }}
+      _active={{
+        bg: 'hoverBg',
+        color: 'brandText',
+      }}
+      transition="all .2s ease"
+      position="relative"
+      gap="2"
+    >
+      {typeof item.icon === 'string' ? (
+        <Image src={item.icon} boxSize="6" alt="icon" />
+      ) : (
+        <Icon as={item.icon} boxSize="6" />
+      )}
+      {t(item.label)}
+      {isActiveRoute && (
+        <NavIndicator position="absolute" right="0" bg="transparent" />
+      )}
+    </Flex>
+  )
 
   return (
     <LinkBox {...rest}>
-      <LinkOverlay href={item.route} target={item.target} onClick={onClose}>
-        <Flex
-          h="40px"
-          align="center"
-          pl={{ base: 5, lg: '15' }}
-          cursor="pointer"
-          data-active={dataAttr(isActiveRoute)}
-          color="grayText"
-          fontWeight="medium"
-          _hover={{
-            bg: 'divider',
-            color: 'dimmedText',
-          }}
-          _active={{
-            bg: 'hoverBg',
-            color: 'brandText',
-          }}
-          transition="all .2s ease"
-          position="relative"
-          gap="2"
+      {isExternalLink ? (
+        <Link
+          href={item.route}
+          target="_blank"
+          rel="noopener noreferrer"
+          onClick={onClose}
         >
-          {typeof item.icon === 'string' ? (
-            <Image src={item.icon} boxSize="6" alt="icon" />
-          ) : (
-            <Icon as={item.icon} boxSize="6" />
-          )}
-          {t(item.label)}
-          {isActiveRoute && (
-            <NavIndicator position="absolute" right="0" bg="transparent" />
-          )}
-        </Flex>
-      </LinkOverlay>
+          {renderLinkContent()}
+        </Link>
+      ) : (
+        <LinkOverlay href={item.route} target={item.target} onClick={onClose}>
+          {renderLinkContent()}
+        </LinkOverlay>
+      )}
     </LinkBox>
   )
 }