Update public/index.js

public/index.js

@@ -228,41 +228,22 @@ if (navMenu) {
 }
 
 document.querySelectorAll('.nav-link').forEach(link => {
-    link.addEventListener('click', (e) => {
-        const href = link.getAttribute('href');
-        if (href && !href.startsWith('#')) {
-            closeMenu();
-        } else if (href && href.startsWith('#')) {
-            e.preventDefault();
-            const target = document.querySelector(href);
-            if (target) {
-                closeMenu();
-                setTimeout(() => {
-                    target.scrollIntoView({ behavior: 'smooth', block: 'start' });
-                }, 400);
-            }
-        }
-    });
+    link.addEventListener('click', handleNavLinkClick);
 });
 
 document.querySelectorAll('a[href^="#"]').forEach(anchor => {
-    anchor.addEventListener('click', function (e) {
-        const href = this.getAttribute('href');
-        if (!href || href === '#') return;
-        
-        e.preventDefault();
-        const target = document.querySelector(href);
-        if (target) {
-            if (navMenu && navMenu.classList.contains('active')) {
-                closeMenu();
-                setTimeout(() => {
-                    target.scrollIntoView({ behavior: 'smooth', block: 'start' });
-                }, 400);
-            } else {
+    if (!anchor.classList.contains('nav-link')) {
+        anchor.addEventListener('click', function (e) {
+            const href = this.getAttribute('href');
+            if (!href || href === '#') return;
+            
+            e.preventDefault();
+            const target = document.querySelector(href);
+            if (target) {
                 target.scrollIntoView({ behavior: 'smooth', block: 'start' });
             }
-        }
-    });
+        });
+    }
 });
 
 window.addEventListener('load', () => {
@@ -294,13 +275,56 @@ function checkAuthStatus() {
             navMenuEl.innerHTML = `
                 <a href="/dashboard" class="nav-link"><i class="fas fa-chart-line"></i> Dashboard</a>
                 <a href="/profile" class="nav-link"><i class="fas fa-user"></i> Profile</a>
-                <a href="#" onclick="logout()" class="nav-link"><i class="fas fa-sign-out-alt"></i> Logout</a>
+                <a href="#" class="nav-link" id="logoutBtn"><i class="fas fa-sign-out-alt"></i> Logout</a>
             `;
             
-            document.querySelectorAll('.nav-link').forEach(link => {
-                link.addEventListener('click', closeMenu);
-            });
+            attachNavLinkListeners();
+        }
+    } else {
+        attachNavLinkListeners();
+    }
+}
+
+function attachNavLinkListeners() {
+    document.querySelectorAll('.nav-link').forEach(link => {
+        link.removeEventListener('click', handleNavLinkClick);
+        link.addEventListener('click', handleNavLinkClick);
+    });
+    
+    const logoutBtn = document.getElementById('logoutBtn');
+    if (logoutBtn) {
+        logoutBtn.addEventListener('click', (e) => {
+            e.preventDefault();
+            closeMenu();
+            setTimeout(() => {
+                logout();
+            }, 400);
+        });
+    }
+}
+
+function handleNavLinkClick(e) {
+    const href = this.getAttribute('href');
+    
+    if (!href || href === '#' || this.id === 'logoutBtn') {
+        return;
+    }
+    
+    if (href.startsWith('#')) {
+        e.preventDefault();
+        const target = document.querySelector(href);
+        if (target) {
+            closeMenu();
+            setTimeout(() => {
+                target.scrollIntoView({ behavior: 'smooth', block: 'start' });
+            }, 400);
         }
+    } else {
+        e.preventDefault();
+        closeMenu();
+        setTimeout(() => {
+            window.location.href = href;
+        }, 400);
     }
 }
 
@@ -538,6 +562,7 @@ document.addEventListener('DOMContentLoaded', function() {
     preventHorizontalScroll();
     loadStats();
     checkAuthStatus();
+    attachNavLinkListeners();
     setInterval(loadStats, 30000);
 
     setTimeout(() => {