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

Align with wcag spec #147

Open
oliverfoster opened this issue Apr 25, 2022 · 5 comments · May be fixed by #198
Open

Align with wcag spec #147

oliverfoster opened this issue Apr 25, 2022 · 5 comments · May be fixed by #198
Assignees

Comments

@oliverfoster
Copy link
Member

Recommendation https://www.w3.org/TR/wai-aria-practices-1.2/examples/combobox/combobox-select-only.html

@joe-replin
Copy link
Contributor

Bug Description:

In the Matching component, each drop-down is labeled only using its selected value. There is no indication when the field is focused, what the corresponding question for the field is.

Steps to Reproduce:

Using a screen reader, open the course prototype.
Open the standard experience, then locate the Matching section
Focus on one of the drop-down menus and note the control's label. 

Actual Behavior:

Currently, the selected value is the only means of identifying the dorp-down menus.

Expected Behavior:

We recommend the Select only combobox structure rather than a button menu to better match the expected behavior of the Matching form fields :​https://www.w3.org/TR/wai-aria-practices-1.2/examples/combobox/combobox-select-only.html

This component will allow users to associate the question with the combobox as label.

@chris-steele
Copy link
Contributor

@joe-replin that link seems to be broken, is this equivalent? https://w3c.github.io/aria-practices/examples/combobox/combobox-select-only.html

@joe-replin
Copy link
Contributor

Hey @chris-steele, the link in my post seems to be linking correctly for me now.

@joe-replin
Copy link
Contributor

@chris-steele, @oliverfoster, raising this issue for awareness to see if we were still planning on refactoring this component to make use of the combobox. Understand it used to have one and it was dropped in favor for better visual styling.

@joe-replin
Copy link
Contributor

Select dropdowns in the Matching component do not clearly communicate the following:

  1. A meaningful label. Currently the selected value serves as the field label.
  2. A combobox role

Missing or inaccurate name and role information impact a screen reader users' understanding of the purpose and expected behavior of a control.

We recommend referring to the updated "Select Only Combobox" component from the W3C: https://www.w3.org/WAI/ARIA/apg/patterns/combobox/examples/combobox-select-only/

@kirsty-hames kirsty-hames self-assigned this Sep 5, 2024
github-actions bot pushed a commit that referenced this issue Nov 11, 2024
# [7.6.0](v7.5.0...v7.6.0) (2024-11-11)

### Update

* Added dropdown a11y labeling (refs #147) (#192) ([e921b28](e921b28)), closes [#147](#147) [#192](#192)
@oliverfoster oliverfoster linked a pull request Nov 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Reviewing
Development

Successfully merging a pull request may close this issue.

5 participants