Skip to content

dialog box add aria-label#77095

Merged
isidorn merged 4 commits intomasterfrom
isidorn/dialogLabel
Jul 31, 2019
Merged

dialog box add aria-label#77095
isidorn merged 4 commits intomasterfrom
isidorn/dialogLabel

Conversation

@isidorn
Copy link
Collaborator

@isidorn isidorn commented Jul 10, 2019

fixes #70981

This PR adds the aria-label to the custom dialog box.
I have tested this on my Mac and works nicely.

If you think this is ok please merge since next two weeks I am out.

@isidorn isidorn added this to the July 2019 milestone Jul 10, 2019
@isidorn isidorn requested a review from sbatten July 10, 2019 11:53
Copy link
Member

@sbatten sbatten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@isidorn should this label contain both the message and detail text?

message: Do you want to save the changes you made to dialog.ts?
detail: Your changes will be lost if you don't save them.

@isidorn
Copy link
Collaborator Author

isidorn commented Jul 31, 2019

@sbatten even though we could add it, I think it would not be concise.
And most of the feedback we got from the accessibility community is to make the messages concise and clear.
So let's go with the current aproach, and we can change it if we recieve feedback.

Thanks!

@isidorn isidorn merged commit 85ddb90 into master Jul 31, 2019
@isidorn isidorn deleted the isidorn/dialogLabel branch July 31, 2019 13:37
@github-actions github-actions bot locked and limited conversation to collaborators Mar 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ARIA info to dialogs

3 participants