-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add .radio-group
component
#979
Conversation
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/primer/primer-css/ql4780xew |
Unchanged from dotcom
This seems un-used
Was thinking of also making this a "checkbox group". Maybe without showing the checkboxes? For use cases where multiple selections should be possible: And would be nice if the radio/checkbox can also be inside the Maybe one day ™️ ? |
Ok, I'll count this as a 👍 and merge it. 🚢 |
This adds the
.radio-group
component from dotcom.📖 Docs Preview
TODO
TODO on dotcom
radio-group.scss
https://github.com/github/github/pull/128984/cc @primer/ds-core