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

[Dropdown] Selection should be announced to screenreader #6523

Open
wants to merge 7 commits into
base: next
Choose a base branch
from

Conversation

limograf
Copy link

@limograf limograf commented Aug 6, 2018

Description

made div holding current selection a live region so the selection is announced by screenreader.

Testcase

Navigate to a select in a form using a screenreader (even Chromevox)
Screenreader is silent, cannot traverse options, cannot select options, form is broken

My fix is described above

jlukic and others added 7 commits July 8, 2018 17:05
made div holding current selection a live region so the selection is announced by screenreader. Would also live to label the created input that precedes it on search/select component (wherever that may be), which is currently silent and misleads the user.
@y0hami y0hami changed the base branch from master to next August 7, 2018 07:41
@limograf
Copy link
Author

limograf commented Aug 20, 2018

Please do come back to me with any queries on this. Eager to get some access into SUI so I can use the main build for my institutional clients. At the moment the framework is unusable as it fails basic access requirements. Would also like to, at minimum:

  • dropdown: label the created input that precedes it on search/select component (wherever that may be), which is currently silent and misleads the user.
  • supply you with a basic wai-aria kitchen sink you can include in your site build
  • fix up the modal a bit to manage focus

@Atulin
Copy link

Atulin commented Dec 12, 2018

I think you should consider creating a PR on fomantic/Fomantic-UI if you have any enhancements in mind. SUI is a little bit on the dead side nowadays.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants