-
Notifications
You must be signed in to change notification settings - Fork 0
Dialog accessibility improvements #103
Comments
@JoshBeveridge thanks for writing the requirements document for the Dialog component. I reviewed it and had few questions in regards to our implementation of it:
Also, if there are any other improvements that I have been missed, please make note in this issue, thanks. |
@mnigh I'm not sure if @yonikid15 passed this along to you, but the a11y issues with the dialog weren't just limited to the requirements in Figma (which will need to be updated once we figure out the other work required here). To answer your two questions:
I think the bigger issue with the dialogs is the way screen readers are accessing and navigating the content itself (hence my comment on the |
|
After chatting with @mnigh and @esizer, we settled on the following:
Given these things, we talked about doing the following:
|
Description
Using Josh's Dialog requirements, make improvements to the Dialog component to improve accessibility and UI.
The text was updated successfully, but these errors were encountered: