Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: enhance accessibility for menu items with keyboard support #2655

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ImBIOS
Copy link

@ImBIOS ImBIOS commented Feb 7, 2025

What/Why/How?

Fix #2621

Reference

null

Tests

  • tested manually and locally.
  • unit test not included due to the component need a lot of code to write to jsut write test for this one function, can be added as additional task.

Screenshots (optional)

loom with keyboard stroke visualizer might makes more sense

Check yourself

  • Code is linted
  • Tested
  • All new/updated code is covered with tests

@ImBIOS ImBIOS requested a review from a team as a code owner February 7, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TOC sections cannot be toggled with keyboard
1 participant