hover effect on subnav item

Esri · Dec 17, 2024 · c518363 · c518363
1 parent 4cb7309
commit c518363
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/eds/blocks/local-navigation/local-navigation.css b/eds/blocks/local-navigation/local-navigation.css
@@ -53,7 +53,7 @@
   padding-inline: var(--space-6);
 }
 
-.local-navigation ul li a[aria-current='true']:hover {
+.local-navigation ul li a[aria-current='true']:hover, .local-navigation ul li a:hover {
   background-color: #2b2b2b;
 }
 

diff --git a/eds/blocks/local-navigation/local-navigation.js b/eds/blocks/local-navigation/local-navigation.js
@@ -53,13 +53,12 @@ function listenSubNav(subNavItems) {
   });
 
   document.addEventListener('click', (e) => {
-    let isClickInside = subNavItems.contains(e.target);
-
-    if (!isClickInside) { 
+    const isClickInside = subNavItems.contains(e.target);
+    if (!isClickInside) {
       subNavItems.setAttribute('aria-expanded', 'false');
       subNavItems.nextElementSibling.setAttribute('aria-hidden', 'true');
     }
-});
+  });
 }
 
 /**
@@ -77,7 +76,7 @@ function appendPageTitle(pgObj, block, i, menuTitle) {
   aHref.innerHTML = `${pgObj.pageTitle}`;
   aHref.setAttribute('aria-current', 'false');
 
-  if ((currPageTitle === pgObj.pageTitle.toLowerCase() || menuTitle === pgObj.pageTitle) && !pgObj.subnavItems) {
+  if ((currPageTitle === pgObj.pageTitle.toLowerCase() || menuTitle === pgObj.pageTitle)) {
     aHref.setAttribute('aria-current', 'true');
   }