From a02554f45f1a55cfd46eeb07e6abdd5d8aa5c08b Mon Sep 17 00:00:00 2001
From: dimasma0305 <dimasmaulana0305@gmail.com>
Date: Wed, 15 Jan 2025 12:17:02 +0800
Subject: [PATCH] feat: refactor Navbar component to use dynamic navigation
 links and button properties for improved maintainability

---
 src/app/_components/navbar.tsx | 54 ++++++++++++++++------------------
 1 file changed, 25 insertions(+), 29 deletions(-)

diff --git a/src/app/_components/navbar.tsx b/src/app/_components/navbar.tsx
index 7c35219..876ba49 100644
--- a/src/app/_components/navbar.tsx
+++ b/src/app/_components/navbar.tsx
@@ -11,42 +11,42 @@ import Button from './button';
 const Navbar: React.FC = () => {
   const [isMobileMenuOpen, setIsMobileMenuOpen] = useState(false);
 
+  const navLinks = [
+    { href: "/", text: "Home" },
+    { href: "/ctfs", text: "CTF's" },
+    { href: "/repositories", text: "Repositories" },
+    { href: "/indonesia-ctf-2025", text: "INDONESIA CTF 2025" },
+    { href: "/mobile-ctf-2025", text: "Mobile CTF 2025" },
+  ];
+
+  const buttonProps = {
+    text: 'Join Playground',
+    href: 'https://playground.tcp1p.team/',
+  };
+
   return (
     <nav className="fixed top-0 left-0 right-0 bg-gray-900/80 backdrop-blur-lg border-b border-red-500/10 z-50">
       <div className="max-w-7xl mx-auto">
         <div className="flex items-center justify-between h-16 px-4 sm:px-6 lg:px-8">
           <div className="flex-shrink-0">
             <div className="flex items-center space-x-2">
-                <Image src={logo} alt={"TCP1P Logo"} width={120}></Image>
+              <Image src={logo} alt={"TCP1P Logo"} width={120}></Image>
             </div>
           </div>
 
           <div className="hidden md:flex items-center justify-center space-x-8">
-            <Link href="/" className="nav-item text-gray-300 hover:text-red-400 transition-colors duration-300">
-              Home
-            </Link>
-            <Link href="/ctfs" className="nav-item text-gray-300 hover:text-red-400 transition-colors duration-300">
-              CTF&apos;s
-            </Link>
-            <Link href="/repositories" className="nav-item text-gray-300 hover:text-red-400 transition-colors duration-300">
-              Repositories
-            </Link>
-            <Link href="/indonesia-ctf-2025" className="nav-item text-gray-300 hover:text-red-400 transition-colors duration-300">
-              INDONESIA CTF 2025
-            </Link>
-            <Link href="/mobile-ctf-2025" className="nav-item text-gray-300 hover:text-red-400 transition-colors duration-300">
-              Mobile CTF 2025
-            </Link>
+            {navLinks.map((link, index) => (
+              <Link key={index} href={link.href} className="nav-item text-gray-300 hover:text-red-400 transition-colors duration-300">
+                {link.text}
+              </Link>
+            ))}
           </div>
 
           <div className="hidden md:block">
-            <Button text='Join Playground' href='https://playground.tcp1p.team/' target='_blank'></Button>
+            <Button text={buttonProps.text} href={buttonProps.href} target='_blank'></Button>
           </div>
 
           <div className="md:hidden flex items-center space-x-4">
-            {/* <button className="px-4 py-1.5 bg-gradient-to-r from-red-600 to-red-500 text-white text-sm rounded-full font-medium hover:from-red-500 hover:to-red-400 transition-all duration-300 shadow-lg shadow-red-500/30">
-              Get Started
-            </button> */}
             <button
               onClick={() => setIsMobileMenuOpen(!isMobileMenuOpen)}
               className="mobile-menu-button p-2 rounded-md text-red-300 hover:bg-red-900/30"
@@ -57,15 +57,11 @@ const Navbar: React.FC = () => {
         </div>
 
         <div className={`mobile-menu md:hidden px-4 pb-4 ${isMobileMenuOpen ? '' : 'hidden'}`}>
-          <Link href="/" className="block px-4 py-2 text-gray-300 hover:text-red-400 transition-colors duration-300">
-            Home
-          </Link>
-          <Link href="/ctfs" className="block px-4 py-2 text-gray-300 hover:text-red-400 transition-colors duration-300">
-            CTF&apos;s
-          </Link>
-          <Link href="/repositories" className="block px-4 py-2 text-gray-300 hover:text-red-400 transition-colors duration-300">
-            Repositories
-          </Link>
+          {navLinks.map((link, index) => (
+            <Link key={index} href={link.href} className="block px-4 py-2 text-gray-300 hover:text-red-400 transition-colors duration-300">
+              {link.text}
+            </Link>
+          ))}
         </div>
       </div>
     </nav>