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: incorrectly behavior when click radio label #160

Merged
merged 1 commit into from
Jul 28, 2024

Conversation

lawvs
Copy link
Contributor

@lawvs lawvs commented Jul 28, 2024

This pull request fixes an issue where the radio label was not behaving correctly when clicked.

The issue was caused by incorrect usage of the id and htmlFor attribute in the Radio components.

This PR updates components to use the useId within Radio and respect the ID input by the user.

Before

Screen.Recording.2024-07-28.at.18.02.26.mov

After

Screen.Recording.2024-07-28.at.18.03.20.mov

Copy link

vercel bot commented Jul 28, 2024

@lawvs is attempting to deploy a commit to the RSS3 Team on Vercel.

A member of the Team first needs to authorize it.

@lawvs lawvs changed the title Fix incorrect behavior when clicking radio label fix: incorrectly behavior when click radio label Jul 28, 2024
Copy link
Member

@Innei Innei left a comment

Choose a reason for hiding this comment

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

Thank you!

@Innei Innei merged commit 5ff0d3d into RSSNext:main Jul 28, 2024
1 of 2 checks passed
@lawvs lawvs deleted the fix/radio-label branch July 28, 2024 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants