fix: use className prop in components where it was unused#735
fix: use className prop in components where it was unused#735Valley-15 wants to merge 2 commits intocosscom:mainfrom
Conversation
|
@Valley-15 is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
pasqualevitiello
left a comment
There was a problem hiding this comment.
Hey, thanks for the contribution! We just need two quick fixes before moving forward:
Scope: Could you revert the changes in toggle-group.tsx? We try to keep PRs focused on one task to make auditing easier.
The Registry Flow: Since we use a registry system, changes need to be made in the source files at apps/ui/registry/default/ui. Once you've made your edits, run this command string from apps/ui to update the generated components:
bun format:all && bun registry:build && bun ui:sync
You can find the full breakdown of the registry architecture in the CONTRIBUTING.md.
Summary
Ensured that
classNameprop is properly used in CommandPanel, DrawerViewport, and SheetViewport components.Fixes
Fixes #733