Skip to content

Feature/Checkbox#108

Open
alexxuyaowen wants to merge 11 commits intopatricklindev:masterfrom
alexxuyaowen:feature/ByAlex/Checkbox
Open

Feature/Checkbox#108
alexxuyaowen wants to merge 11 commits intopatricklindev:masterfrom
alexxuyaowen:feature/ByAlex/Checkbox

Conversation

@alexxuyaowen
Copy link
Copy Markdown
Collaborator

No description provided.

@alexxuyaowen alexxuyaowen changed the title first commit WIP: Feature/ByAlex/Checkbox Jul 8, 2022
@alexxuyaowen alexxuyaowen changed the title WIP: Feature/ByAlex/Checkbox WIP: Feature/Checkbox Jul 8, 2022
Copy link
Copy Markdown

@nickchow2020 nickchow2020 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

almost done, you just missing several requirements

Comment on lines +9 to +15
defaultChecked?: boolean;
disabled?: boolean;
checked?: boolean;
label?: string;
size?: CheckboxSize;
className?: string;
color?: CheckboxColor;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First I check the requirement for Checkbox, User should be able to pass the Icons from prop but i don't see any implementation for it

Second requirement for this assignment is that user able to pass a onChange Callback function as prop and I don't see it as well

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Comment on lines +6 to +9
it('should match snapshot', () => {
const { asFragment } = render(<Checkbox />);
expect(asFragment()).toMatchSnapshot();
});
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can add more testing to it

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

@alexxuyaowen alexxuyaowen changed the title WIP: Feature/Checkbox Feature/Checkbox Jul 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants