fix (a11y): add aria hidden to navigation flyout when visually hidden (…

…#811) * fix (a11y): add aria hidden to NavigationFlyout when visually hidden * fix (a11y): remove unnecessary aria-hidden from background overlay --------- Co-authored-by: noa.santo <[email protected]>
satellytes · Sep 18, 2024 · 5fcee62 · 5fcee62
1 parent 1e0ab8c
commit 5fcee62
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/components/layout/header/menu-flyout.tsx b/src/components/layout/header/menu-flyout.tsx
@@ -65,7 +65,11 @@ interface NavigationFlyoutProp {
 export const NavigationFlyout: React.FC<NavigationFlyoutProp> = (props) => {
   return (
     <>
-      <FullscreenOverlay $visible={props.visible} onClick={props.onClick}>
+      <FullscreenOverlay
+        $visible={props.visible}
+        onClick={props.onClick}
+        aria-hidden={!props.visible}
+      >
         <ScrollContainer>
           <FullHeightNavigation
             translation={props.translation}