Replies: 1 comment
-
I did have setup once. My issue with it is it adds an install step to those coming over from shadcn based setup/install, but maybe thats easier enough to understand if I document it. Plus it would become apparent quickly that something is missing. I think If I had set-up my own CLI that followed or used shadcn's and then applied some migration code, this would work really well as well. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Currently, most components call
composeRenderProps
withcn
to form the className. Wondering if we can introduceacn
which merges the functionality of the two.We can define
acn
as follows:Alternatively, if we do not wish to introduce a separate utility function, we can just make modify
cn
instead so that it ends up with the following signature:This alternative
cn
will need to do more work at runtime though -- and so personally prefer keeping them separate.I can open a pull req implementing any of the two approaches.
Beta Was this translation helpful? Give feedback.
All reactions