-
Notifications
You must be signed in to change notification settings - Fork 304
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
docs: adding contribution guide #413
Conversation
### CSS Classes | ||
There are cases when we want to accept `class` as a prop in our `shadcn/vue` components and then combine it with a default style on our `radix-vue` component. | ||
|
||
In these cases, we can not use `v-bind` while `class` is declared as a prop, because this would lead in [double class binding](https://github.com/radix-vue/shadcn-vue/pull/241), so the `class` propert must be dealt specefically. |
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.
In these cases, we can not use
v-bind
whileclass
is declared as a prop, because this would lead in double class binding, so theclass
propert must be dealt specefically.
We can use v-bind
when class is declared as prop
we only use this way to prevent class duplication with cn
function
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.
Yup. As long as we have class
as prop, we can use v-bind
😁
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.
Thanks for the feedback ! I tried to be more clear on this one.
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 also added some additional information based on CONTRIBUTING.md
.
This is so nice Saeid 💯 Also I would add these changes to |
Some information in |
Please edit or add any other parts that you see fit.
resolves #409.