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

Update Guidance for RadioButtonGroup #4066

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

luketa8
Copy link
Collaborator

@luketa8 luketa8 commented Jul 26, 2024

Deploy Preview

Issue

What does this PR do?

Update guidance for RadioButtonGroup clarifying its usage for filtering

Where should the reviewer start?

Clear selection discovery
Clear selection Figma

@luketa8 luketa8 changed the title Update radiobuttongroup.mdx Update Guidance for RadioButtonGroup and Select Jul 26, 2024
@luketa8 luketa8 changed the title Update Guidance for RadioButtonGroup and Select Update Guidance for RadioButtonGroup Jul 30, 2024
@luketa8 luketa8 self-assigned this Jul 30, 2024
@luketa8 luketa8 requested a review from julauxen July 30, 2024 20:21
Copy link
Collaborator

@taysea taysea left a comment

Choose a reason for hiding this comment

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

Tiny comments but let's not let it hold up moving forward with the PR.

Copy link
Collaborator

@julauxen julauxen left a comment

Choose a reason for hiding this comment

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

I added a minor comment, but I believe once that is clarified we can merge. This was supposed to be a "quick fix" and the main issue (the filtering scenario) is now clearly explained.

@@ -29,9 +29,11 @@ import {

### When to use

RadioButtonGroup is used to lay out a discreet list of mutually related options that are easily visible. RadioButtonGroup requires an input choice, so it is important to be clear with what is being asked of the user on the label.
RadioButtonGroup is used to lay out a list of distinct, mutually exclusive options that are easily visible. Since a selection is required, it’s important to be clear with what is being asked of the user on the label.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am guessing the phrase "since a selection is required" means "once an option is selected, it does not allow for deselection" so I would suggest:
Once an option is selected it does not allow for deselection, so it’s important to be clear with what is being asked of the user on the label.

@luketa8 is this the right interpretation?

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.

3 participants