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

feat(SIP section menu): Update SIP section menu #29

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

delisma
Copy link
Contributor

@delisma delisma commented Mar 27, 2020

Add keyboard accessibility
Update hover state for active menu element
Add ARIA roles

Add keyboard accessibility
Update hover state for active menu element
Add ARIA roles
@delisma delisma requested a review from Borges-Porteous March 27, 2020 11:55
@delisma delisma self-assigned this Mar 27, 2020
@delisma delisma added the enhancement New feature or request label Mar 27, 2020
@delisma
Copy link
Contributor Author

delisma commented Mar 27, 2020

@Borges-Porteous Still a few things missing to make it fully accessible:

  • First tab should go straight to the active element by default not the first item on the list
  • esc should remove focus and/or move to the next navigation block on the page

@delisma delisma force-pushed the master branch 8 times, most recently from 16a78da to 3fa6591 Compare October 16, 2020 22:48
@delisma delisma force-pushed the master branch 2 times, most recently from 552c89a to 1ec6588 Compare January 6, 2021 23:29
@delisma delisma marked this pull request as draft June 26, 2023 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant