-
-
Notifications
You must be signed in to change notification settings - Fork 7.4k
feat(components): changed sonner defaults to use lucide icons #7620
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
Conversation
|
@jguddas is attempting to deploy a commit to the shadcn-pro Team on Vercel. A member of the Team first needs to authorize it. |
shadcn
left a comment
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 like this. Can we make the change to apps/v4/registry/new-york-v4/ui/sonner.tsx instead?
(I want to make all updates to the new components and then potentially backport to the v3 versions.)
# Conflicts: # apps/v4/public/r/styles/new-york-v4/registry.json # apps/v4/public/r/styles/new-york-v4/sonner.json # apps/v4/registry.json # apps/v4/registry/__index__.tsx # apps/v4/registry/registry-examples.ts # apps/www/public/r/styles/new-york-v4/registry.json # apps/www/public/r/styles/new-york-v4/sonner.json
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
shadcn
left a comment
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.
Thank you.
…-ui#7620) * feat(components): changed sonner defaults to use lucide icons * Update new-york-v4 sonner.tsx * fix: icons and docs * fix * fix --------- Co-authored-by: shadcn <[email protected]>
To avoid mixing icon sets, we can provide lucide icons to sonner as nice cohesive defaults.
Before:

After:
