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

Akmal / feat: add radio button & radio group components #58

Merged
merged 21 commits into from
Apr 15, 2024

Conversation

akmal-deriv
Copy link
Contributor

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.

Copy link

cloudflare-workers-and-pages bot commented Apr 5, 2024

Deploying quill-ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3b1d969
Status: ✅  Deploy successful!
Preview URL: https://78d6470f.quill-ui.pages.dev
Branch Preview URL: https://f-add-radio-button-component.quill-ui.pages.dev

View logs

<StandaloneCircleDotBoldIcon
data-testid={`dt_checked_icon_${value}_${disabled}`}
fill={disabled ? "#b8b8b8" : "#000000"}
iconSize={size}
Copy link
Contributor

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?

Copy link
Contributor Author

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.

@prince-deriv prince-deriv merged commit c145427 into main Apr 15, 2024
4 checks passed
@prince-deriv prince-deriv deleted the f-add-radio-button-component branch April 15, 2024 10:54
Copy link

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants