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

fix(RadioButton): remove unecessary aria-label #4678

Merged
merged 9 commits into from
Nov 16, 2019
Merged

fix(RadioButton): remove unecessary aria-label #4678

merged 9 commits into from
Nov 16, 2019

Conversation

dakahn
Copy link
Contributor

@dakahn dakahn commented Nov 14, 2019

Closes #4491

Really quick one here. Removes an unneeded aria-label that was causing label duplication in Safari and just totally breaking labeling for VoiceOver in Chrome.

Testing / Reviewing

In Safari with VO make sure that the label is only read once. In Chrome -- check and make sure the label is read.

@dakahn dakahn requested a review from a team as a code owner November 14, 2019 12:27
@dakahn dakahn added this to Pull requests 📋 in Carbon React accessibility via automation Nov 14, 2019
@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for the-carbon-components ready!

Built with commit 80d8f60

https://deploy-preview-4678--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for carbon-elements ready!

Built with commit 80d8f60

https://deploy-preview-4678--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for carbon-components-react ready!

Built with commit 80d8f60

https://deploy-preview-4678--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for carbon-elements ready!

Built with commit 4a439ac

https://deploy-preview-4678--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for carbon-components-react ready!

Built with commit 4a439ac

https://deploy-preview-4678--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for the-carbon-components ready!

Built with commit 4a439ac

https://deploy-preview-4678--the-carbon-components.netlify.com

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

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

it looks like snapshots need to be updated to pass CI

Copy link
Contributor

@abbeyhrt abbeyhrt left a comment

Choose a reason for hiding this comment

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

Once the snapshots are updated, this looks good to me! 🎉

Copy link
Contributor

@aledavila aledavila left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @dakahn!

@asudoh
Copy link
Contributor

asudoh commented Nov 14, 2019

@dakahn Could you update the snapshot? Thanks!

@asudoh asudoh merged commit 77d8363 into carbon-design-system:master Nov 16, 2019
Carbon React accessibility automation moved this from Pull requests to Closed issues Nov 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Carbon React accessibility
Closed issues 🏁
Development

Successfully merging this pull request may close these issues.

RadioButton React - Screen reader cannot read label
6 participants