Skip to content
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: [CheckboxGroup] Issue #465

Open
kevinmitch14 opened this issue Apr 18, 2024 · 0 comments
Open

Docs: [CheckboxGroup] Issue #465

kevinmitch14 opened this issue Apr 18, 2024 · 0 comments
Labels
documentation Improvements or additions to documentation

Comments

@kevinmitch14
Copy link

Should onValueChange() be documented here? It says this component is based on div which doesn't have onValueChange. I was originally using onChange() but this does not work.

I think there are a couple more components that don't have the control props documented. Mainly the new Card/Group components.

Usually, I go to the equivalent primitive docs to find which props/change handlers are supported but when there is no equivalent primitive, it becomes a little confusing to understand what is supported and what isn't.

Context: Using CheckboxGroup in a controlled form with React Hook Form.

@kevinmitch14 kevinmitch14 changed the title [CheckboxGroup] Issue Docs: [CheckboxGroup] Issue Apr 18, 2024
@vladmoroz vladmoroz added the documentation Improvements or additions to documentation label Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants