fix: add error handling for active navigation link processing

friendlylabs · Jan 22, 2025 · c1bc998 · c1bc998
1 parent 7ff67f2
commit c1bc998
Showing 1 changed file with 23 additions and 13 deletions.
diff --git a/public/assets/js/layout.js b/public/assets/js/layout.js
@@ -37,20 +37,30 @@ if (navbarVerticalStyle === 'darker') {
 }
 
 // add active to current page
-var currentUrl = window.location.href;
-var navLinks = document.querySelectorAll('.navbar-nav a');
-navLinks.forEach(function (navLink) {
-    if (navLink.href === currentUrl) {
-        navLink.classList.add('active');
-
-        // check if navLink is in a dropdown
-        var dropdown = navLink.closest('.dropdown');
-        if (dropdown) {
-            // add to dropdown-toggle
-            dropdown.querySelector('.dropdown-toggle').classList.add('active');
+try {
+    var currentUrl = window.location.href;
+    var navLinks = document.querySelectorAll('.navbar-nav a');
+
+    navLinks.forEach(function (navLink) {
+        if (navLink.href === currentUrl) {
+            navLink.classList.add('active');
+
+            // Check if navLink is in a dropdown
+            var dropdown = navLink.closest('.dropdown');
+            if (dropdown) {
+                var dropdownToggle = dropdown.querySelector('.dropdown-toggle');
+                if (dropdownToggle) {
+                    dropdownToggle.classList.add('active');
+                } else {
+                    console.warn('Dropdown found, but no .dropdown-toggle element exists.');
+                }
+            }
         }
-    }
-});
+    });
+} catch (error) {
+    console.error('An error occurred while processing navigation links:', error);
+}
+
 
 function menuToOffcanvas() {
     // check if offCanvas #mobileMenu exists, if not create it