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

Accessible Context Menu #30

Open
zoltan-dulac opened this issue Jun 12, 2023 · 4 comments · May be fixed by #209
Open

Accessible Context Menu #30

zoltan-dulac opened this issue Jun 12, 2023 · 4 comments · May be fixed by #209
Assignees
Labels
L 3 days of effort P2 Medium priority work

Comments

@zoltan-dulac
Copy link
Collaborator

Need to make an accessible version of this: https://www.radix-ui.com/docs/primitives/components/context-menu

Should have visually hidden button to show context menu (or some other method).
Need to be mobile accessible as well. (triple tap? long press?)

@zoltan-dulac zoltan-dulac added L 3 days of effort P2 Medium priority work labels May 31, 2024
@ocjadan
Copy link
Collaborator

ocjadan commented Jul 22, 2024

Hi @zoltan-dulac , I am interested in taking this one up now that my cookie banner pull request is opened. What do you think?

@ocjadan ocjadan self-assigned this Jul 22, 2024
@ocjadan
Copy link
Collaborator

ocjadan commented Jul 23, 2024

First steps, identify the default way to bring up the context menu on different browsers and on mobile. See if this behavior is universal, if the behavior is too fragmented then decide on one for Enable.

@ocjadan
Copy link
Collaborator

ocjadan commented Jul 24, 2024

The command to open the context menu for Safari, Chrome and Firefox when using VoiceOver is ctrl+option+shift+m. Now, looking into ways to customize the context menu.

@ocjadan
Copy link
Collaborator

ocjadan commented Aug 8, 2024

Aug 8, Thursday

  1. Anywhere that has a custom context menu implementation should announce so—i.e., the link should have an aria-describedby that says there's a custom context menu.
  2. Opening the context menu should also announce that arrow key navigation should be used.
  3. Mobile, mobile, mobile. Mobile works?
  4. Test the "menu" button on full-size keyboards. Might be the menu icon button above the numpad on my logi.

@ocjadan ocjadan linked a pull request Aug 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L 3 days of effort P2 Medium priority work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants