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

feat(components): toggle-group #1547

Merged
merged 15 commits into from
Nov 12, 2023
Merged

feat(components): toggle-group #1547

merged 15 commits into from
Nov 12, 2023

Conversation

maxwiseman
Copy link
Contributor

@maxwiseman maxwiseman commented Sep 18, 2023

Toggle-group

Preview

This PR adds the toggle-group primitive from Radix.
Similar to #53 and #610, but with the styling from the existing toggle component.

image

@vercel
Copy link

vercel bot commented Sep 18, 2023

@maxwiseman is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

- Dark mode border color is now consistent with the toggle component
- Added the content field to `components.json` for toggle-group
- Ran build:registry again
@maxwiseman maxwiseman marked this pull request as ready for review September 18, 2023 11:18
@maxwiseman
Copy link
Contributor Author

@shadcn Any updates on whether or not this is going to be merged?

@shadcn
Copy link
Collaborator

shadcn commented Oct 21, 2023

@maxwiseman Looking into it. Note: we have 3 other PRs that also implement the toggle group components:

Copy link

vercel bot commented Nov 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2023 6:58pm

@shadcn shadcn added the area: roadmap This looks great. We'll add it to the roadmap, review and merge. label Nov 12, 2023
@shadcn shadcn mentioned this pull request Nov 12, 2023
Copy link
Collaborator

@shadcn shadcn left a comment

Choose a reason for hiding this comment

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

@maxwiseman I've simplify the implementation a bit and reused the styles from toggle. Thanks for your help on this! Appreciate it!

@shadcn shadcn merged commit 1db90ba into shadcn-ui:main Nov 12, 2023
8 checks passed
@maxwiseman maxwiseman deleted the toggleGroup branch December 2, 2023 21:20
kjxbyz pushed a commit to muse-ui/muse-ui that referenced this pull request Jun 7, 2024
* feat: added toggle-group component

* fix(components): ran build:registry script

* fix(components): fixed colors in toggle-group
- Dark mode border color is now consistent with the toggle component

* fix(components): fixed component.json toggle-group
- Added the content field to `components.json` for toggle-group
- Ran build:registry again

* feat(toggle-group): simplify implementation

---------

Co-authored-by: shadcn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: roadmap This looks great. We'll add it to the roadmap, review and merge. automerge component: toggle group new component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants