Skip to content

Commit

Permalink
A11y markup adjustments - #225
Browse files Browse the repository at this point in the history
  • Loading branch information
joshdavenport committed Aug 10, 2023
1 parent 69e34fc commit 1fa969c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/components/Layout/Header.astro
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ import Navigation from './Header/Navigation.astro';
'md:h-[--logo-height] md:w-[--logo-width]',
'nav-mobile:-ml-10 nav-mobile:h-[--logo-height-mobile] nav-mobile:w-[--logo-width-mobile]',
'nav-mobile-open:invert nav-mobile-open:invert nav-mobile-open:saturate-0 nav-mobile-open:brightness-[0.8] nav-mobile-open:mix-blend-luminosity',
]}>
]}
aria-label="Unitary Fund Homepage">
<Svg name="logo" />
</a>
<Navigation
Expand Down
6 changes: 4 additions & 2 deletions src/components/Layout/Header/Navigation/Menu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,8 @@ export default function Menu({ menu }: NavigationProps) {
'nav-mobile:flex-col',
'nav-desktop:absolute nav-desktop:top-0 nav-desktop:left-0 nav-desktop:flex-wrap nav-desktop:gap-[2px]',
!isOpen && 'hidden',
])}>
])}
role="navigation">
{item.children.map(({ link, text }) => (
<li className="flex grow" key={link}>
<Button
Expand Down Expand Up @@ -160,7 +161,8 @@ export default function Menu({ menu }: NavigationProps) {
className={cn([
'm-0 flex flex-col fixed top-[--logo-height-mobile] left-0 right-0 no-arrow-list',
(!isOpen || !isMobile) && 'hidden',
])}>
])}
role="navigation">
{menuItems}
</ul>
{/* Desktop */}
Expand Down

0 comments on commit 1fa969c

Please sign in to comment.