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(RadioGroup): Radio Group label is now a span to improve a11y #1139

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

silvalaura
Copy link
Collaborator

@silvalaura silvalaura commented Oct 3, 2023

Issue: #

What I did

Label inside RadioGroup is a span instead of a label.

Screenshots:

image

image
image

Checklist

  • changeset has been added
  • Pull request description is descriptive
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes
  • [-] I have added tests that prove my fix is effective or that my feature works

How to test

Confirm a11y passes for RadioGroup

@silvalaura silvalaura requested a review from a team as a code owner October 3, 2023 14:34
@changeset-bot
Copy link

changeset-bot bot commented Oct 3, 2023

🦋 Changeset detected

Latest commit: 2ed9d56

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

@silvalaura silvalaura self-assigned this Oct 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

@silvalaura silvalaura merged commit 1cf3809 into dev Oct 4, 2023
2 checks passed
@silvalaura silvalaura deleted the fix/radioGroupLabel branch October 4, 2023 20:22
@silvalaura silvalaura added the a11y Issues related to accessibility label Oct 4, 2023
@chris-cedrone-cengage
Copy link
Collaborator

Looks good here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Issues related to accessibility
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants