From 2831cabec975a856a47fc5c9ee69f7a3c0db1cec Mon Sep 17 00:00:00 2001 From: FrankZamora Date: Wed, 26 Nov 2025 23:49:10 -0600 Subject: [PATCH] Fix: ROI_Bootstrap_Nav_Walker - allow dropdown links with URLs to navigate - Apply same fix to NavbarRenderer's walker class - Only add data-bs-toggle=dropdown for items without real URL - Fixes Buscador General link navigation --- Public/Navbar/Infrastructure/Ui/NavbarRenderer.php | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Public/Navbar/Infrastructure/Ui/NavbarRenderer.php b/Public/Navbar/Infrastructure/Ui/NavbarRenderer.php index 34f585bf..396f1492 100644 --- a/Public/Navbar/Infrastructure/Ui/NavbarRenderer.php +++ b/Public/Navbar/Infrastructure/Ui/NavbarRenderer.php @@ -328,7 +328,12 @@ class ROI_Bootstrap_Nav_Walker extends Walker_Nav_Menu $atts['class'] = 'nav-link'; if ($args->walker->has_children) { $atts['class'] .= ' dropdown-toggle'; - $atts['data-bs-toggle'] = 'dropdown'; + // Only add data-bs-toggle if no real URL (allows click navigation on desktop) + // CSS hover handles showing dropdown, data-bs-toggle only needed for mobile + $url = !empty($item->url) ? $item->url : ''; + if (empty($url) || $url === '#' || $url === '#!') { + $atts['data-bs-toggle'] = 'dropdown'; + } $atts['role'] = 'button'; $atts['aria-expanded'] = 'false'; }