-
Notifications
You must be signed in to change notification settings - Fork 0
Checkbox component #120
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
Checkbox component #120
Conversation
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.
Copilot reviewed 11 out of 12 changed files in this pull request and generated no comments.
Files not reviewed (1)
- package.json: Language not supported
Comments suppressed due to low confidence (1)
src/pages/Components/components/CheckboxComponents.tsx:93
- The description incorrectly refers to the 'Input' component instead of the 'Checkbox' component. Please update the text to accurately reflect that this example demonstrates the Checkbox component.
The <span className="font-mono font-bold">Input</span> component renders a HTML input element. It is used to capture a single line of text input. The Input component internally uses the Label, HelpText, and FieldError components.
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.
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- package.json: Language not supported
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.
Copilot reviewed 12 out of 13 changed files in this pull request and generated no comments.
Files not reviewed (1)
- package.json: Language not supported
Comments suppressed due to low confidence (1)
src/pages/Components/components/tests/CheckboxComponents.test.tsx:47
- [nitpick] The validation error element is asserted twice in this test (once inside waitFor and immediately after). Consider removing the duplicate assertion to improve clarity and reduce redundancy.
expect(screen.getByTestId('checkbox-terms-error')).toBeDefined();
📢 Instructions
🔧 Change Summary
Describe the changes included in this pull request. Link to the associated GitHub issue(s).
CheckboxcomponentCheckboxexamples📝 Checklist
Pull request authors must complete the following tasks before marking the PR as ready to review.
🧪 Steps to Test
Describe the process to test the changes in this pull request.
Review the checkbox stories in Storybook. In the app...
🔗 Additional Information
Optionally, provide additional details, screenshots, or URLs that may assist the reviewer.