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

Command Palette: combobox suggestions are not announced #50846

Open
afercia opened this issue May 22, 2023 · 10 comments · Fixed by #52930
Open

Command Palette: combobox suggestions are not announced #50846

afercia opened this issue May 22, 2023 · 10 comments · Fixed by #52930
Assignees
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Commands /packages/commands [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@afercia
Copy link
Contributor

afercia commented May 22, 2023

Description

The Command Center implements the combobox ARIA pattern, which is basically a search input combined with a list of suggestions. There are already a few combobox implementations in the editor. This specific implementation uses the cmdk external package.

Unfortunately, right now the combobox provided by cmdk is not accessible. It misses the aria-activedescendant property. This change upstream pacocoursey/cmdk#108 should fix it but it hasn't been released yet. Hopefully, it will be released before the next WordPress release so that there will be a chance to update the package. However, the current cmdk version 0.2.0 is not accessible. As such, the Command Center isn't accessible right now and I'm a bit surprised cmdk has been picked up with no prior accessibility evaluation.

There are a few more a11y issues in the Command Center implementation but it's a bit pointless running a full audit if aria-activedescendant doesn't get fixed first.

Step-by-step reproduction instructions

  • Use Safari + VoiceOver.
  • Go to the Site Editor.
  • Open the Command Center.
  • Search for some term to get a list of suggestions.
  • Use the Down and Up arrow keys to go through the list of suggestions.
  • Observe that VoiceOver doesn't announce the suggestions.

Note: I haven't tested on Windows. Worth noting that for some screen readers the aria-selected=true property set on the highlighted suggestion may be enough to make the announcement work as expected, but for a full support the missing aria-activedescrendant property is required.

Screenshots, screen recording, code snippet

No response

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@afercia afercia added [Type] Bug An existing feature does not function as intended [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Commands /packages/commands labels May 22, 2023
@afercia
Copy link
Contributor Author

afercia commented May 22, 2023

Important note: it appears the cmdk demo at https://cmdk.paco.me does use aria-activedescendant. I guess that demo is based on the development branch of cmdk. However, it still doesn't work, at least with Safari. The suggestions are not announced. There must be something else that breaks the expected ARIA pattern even when aria-activedescendant is fixed.

@afercia
Copy link
Contributor Author

afercia commented Jul 21, 2023

Ping for @alexstine 🙂

@alexstine
Copy link
Contributor

@afercia Thanks for the ping but there's nothing we can do if it is upstream.

However, the current cmdk version 0.2.0 is not accessible. As such, the Command Center isn't accessible right now and I'm a bit surprised cmdk has been picked up with no prior accessibility evaluation.

This is a big problem in Gutenberg today. Really getting sick of it too be honest. We'll never make anything accessible unless people meet us half way. Not saying it happens this way every time, but it happened here.

CC: @annezazu. These are the things leads should be aware of. Command center being inaccessible from day 1.

@afercia
Copy link
Contributor Author

afercia commented Jul 21, 2023

@alexstine I know. What can I say... I share your frustration.

@annezazu
Copy link
Contributor

@youknowriad mind digging in here to follow up on this feedback? Thank you for flagging, Alex and Andrea.

@afercia
Copy link
Contributor Author

afercia commented Jul 26, 2023

Reopening because this still does not work with Safari + VoiceOver. See conversation starting from #52930 (comment)

Worth noting the test instructions, or the actual testing, in the PR that closed this issue were inaccurate in the part Use Safari + VoiceOver. because it doesn't really work with Safari + VoiceOver.

@alexstine
Copy link
Contributor

Sadly, I had no way to test Voiceover at the moment as my Mac is back home. I figured it would be better than nothing to land a Windows fix but that was not intentional to make Voiceover any less important.

@annezazu
Copy link
Contributor

@ntsekouras pinging you back on this after your recent effort <3

@afercia
Copy link
Contributor Author

afercia commented Jul 27, 2023

I figured it would be better than nothing to land a Windows fix but that was not intentional to make Voiceover any less important.

A fix for Windows already covers a big part of the users and it's very welcome. @alexstine thank you for all your work and commitment ❤️

@ntsekouras
Copy link
Contributor

ntsekouras commented Jul 27, 2023

I figured it would be better than nothing to land a Windows fix

Just to clarify that the fix was apparently for Chrome and Firefox in macOS too. The problem still remains for safari browser. I'll see if I can figure out a fix for that too.

--edit

I tried to add aria-owns and doesn't seem to work. Additionally though I tested urlInput that has these changes and I don't think it works either.. @afercia could you try the urlInput once more to see if it works for you? Thanks!

Reference from Apple dev forums.

@annezazu annezazu changed the title Command Center: combobox suggestions are not announced Command Palette: combobox suggestions are not announced Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Commands /packages/commands [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants