-
Notifications
You must be signed in to change notification settings - Fork 6
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
Akmal / feat: add radio button & radio group components #58
Conversation
Deploying quill-ui with Cloudflare Pages
|
lib/components/SelectionControl/radio/radio-button/radio-button.scss
Outdated
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-group/radio-group.stories.tsx
Outdated
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-button/radio-button.scss
Outdated
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-button/radio-button.stories.tsx
Outdated
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-button/radio-button.stories.tsx
Outdated
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-group/__tests__/radio-group.test.tsx
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-group/radio-group.stories.tsx
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-button/radio-button.scss
Outdated
Show resolved
Hide resolved
lib/components/SelectionControl/radio/radio-button/radio-button.stories.tsx
Show resolved
Hide resolved
<StandaloneCircleDotBoldIcon | ||
data-testid={`dt_checked_icon_${value}_${disabled}`} | ||
fill={disabled ? "#b8b8b8" : "#000000"} | ||
iconSize={size} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this icon doesn't have the tabIndex
and the other one has?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to avoid selecting the selected option. Since there's no point to click the option which is already selected, I only added tabIndex to the icon which is used for non-selected options.
🎉 This PR is included in version 1.7.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This feature introduces the implementation of radio button and radio group components to enhance user interface options within the project. Radio buttons provide users with a selection mechanism for exclusive choices within a group, while radio groups manage the collective behavior of multiple radio buttons.
Radio Button Component: The radio button component is created to allow users to select a single option from a set of mutually exclusive choices. Each radio button represents a distinct choice, and only one option can be selected at a time.
Radio Group Component: The radio group component encapsulates multiple radio buttons that belong to the same group. It ensures that only one radio button within the group can be selected at any given time, enforcing the mutual exclusivity of choices.
These components offer a user-friendly and intuitive way for users to make selections within forms, surveys, or any other interactive elements where exclusive choices are required. By adding radio button and radio group components, the project enhances its usability and provides a consistent and familiar interface for users to interact with.