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
-
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