Skip to content

Commit 70f4b21

Browse files
committed
fix: improve accessibility and visibility of navigation links
1 parent 6b3dd81 commit 70f4b21

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/components/Nav/MainNavLinks.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ export const MainNavLinks = ({
4646
class={styles.toggle}
4747
onClick={handleToggle}
4848
aria-expanded={isOpen}
49+
aria-controls="main-links-list main-cta-list"
4950
aria-label={mobileMenuLabel || "Toggle navigation menu"}
5051
>
5152
<div class={styles.mobileMenuLabel}>
@@ -72,7 +73,7 @@ export const MainNavLinks = ({
7273
}`}
7374
>
7475
{renderLogo()}
75-
<ul>
76+
<ul id="main-links-list" hidden={!isOpen}>
7677
{links.map((link) => (
7778
<li key={link.label}>
7879
<a href={link.url} tabIndex={isOpen ? 0 : -1}>
@@ -81,7 +82,7 @@ export const MainNavLinks = ({
8182
</li>
8283
))}
8384
</ul>
84-
<ul class="flex flex-col gap-[15px]">
85+
<ul id="main-cta-list" class="flex flex-col gap-[15px]" hidden={!isOpen}>
8586
<li>
8687
<a
8788
class={styles.buttonlink}

0 commit comments

Comments
 (0)