fix: remove confusing close label for navigation menu

[aduh95]

Description

As I was trying around navigating the website with VoiceOver, I noticed the current design is kinda confusing:

• when the nav menu is closed, and the user focuses the checkbox, VoiceOver says "Show navigation menu, tickbox, unticked" (so far so good).
• when the nav menu is open, and the user focuses the checkbox, VoiceOver says "Close navigation menu, tickbox, ticked", which may be kind of confusing. If the user is not able to get visual clues to know if the menu is actually currently opened, they might assume the menu is actually closed (since the checkbox is ticked).

TBH I'm not at all confident which version is the correct one, if someone who are a screen reader user could chime in, that'd be much appreciated.
An alternative would be to use a more verbose label (such as "Navigation menu is currently closed, tick the box to open it", and "Navigation menu is currently open, untick the box to close it"), not sure if this is preferable.

Validation

Related Issues

#7427

Check List

• I have read the Contributing Guidelines and made commit messages that follow the guideline.
• I have run npm run format to ensure the code follows the style guide.
• I have run npm run test to check if all tests are passing.
• I have run npx turbo build to check if the website builds without errors.
• I've covered new added functionality with unit tests if necessary.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
nodejs-org	✅ Ready (Inspect)	Visit Preview	Feb 1, 2025 10:10pm