Merge branch 'feature/PASE-2049' of github.com:penske-media-corp/pmc-…

…larva into feature/PASE-2049
penske-media-corp · Feb 4, 2025 · 3f3271d · 3f3271d
2 parents 2baae2e + e8fdeb2
commit 3f3271d
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/packages/larva-js/src/interface/MegaMenu/MegaMenu.js b/packages/larva-js/src/interface/MegaMenu/MegaMenu.js
@@ -1,3 +1,7 @@
+// no-global-active-element not needed for non-React context.
+// https://wordpress.stackexchange.com/a/387511
+/* eslint-disable @wordpress/no-global-active-element */
+
 export default class MegaMenu {
 	constructor( el ) {
 		this.el = el;
@@ -63,15 +67,14 @@ export default class MegaMenu {
 					lastFocusableElement.focus();
 					event.preventDefault();
 				}
-			} else {
+
 				// tab key
-				if (
-					document.activeElement === lastFocusableElement &&
-					firstFocusableElement
-				) {
-					firstFocusableElement.focus();
-					event.preventDefault();
-				}
+			} else if (
+				document.activeElement === lastFocusableElement &&
+				firstFocusableElement
+			) {
+				firstFocusableElement.focus();
+				event.preventDefault();
 			}
 		};
 

diff --git a/packages/larva-js/src/interface/MegaMenu/index.js b/packages/larva-js/src/interface/MegaMenu/index.js
@@ -1,3 +1,7 @@
+// no-global-active-element not needed for non-React context.
+// https://wordpress.stackexchange.com/a/387511
+/* eslint-disable @wordpress/no-global-active-element */
+
 import MegaMenu from './MegaMenu.js';
 
 export default function initMegaMenu() {