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 and CheckBoxGroup can accept children now #5460
Conversation
Signed-off-by: gurkiran_singh <gurkiransinghk@gmail.com>
… object Signed-off-by: gurkiran_singh <gurkiransinghk@gmail.com>
Docs should be updated on grommet-site repo since CheckBox and CheckBoxGroup docs have been transitioned to the new format. See grommet/grommet-site#233 for more info |
Signed-off-by: gurkiran_singh <gurkiransinghk@gmail.com>
Signed-off-by: gurkiran_singh <gurkiransinghk@gmail.com>
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.
Looks good to me
Co-authored-by: Eric Soderberg <eric.soderberg@hpe.com>
… into children_checkbox
Signed-off-by: GurkiranSingh <gurkiransinghk@gmail.com>
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.
very close
@@ -21,6 +21,14 @@ export const doc = (CheckBox) => { | |||
checked: PropTypes.bool | |||
.description('Same as React <input checked={} />') | |||
.defaultValue(false), | |||
children: PropTypes.func.description( |
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.
This change will now need to be made in https://github.com/grommet/grommet-site/blob/master/src/screens/CheckBox.js
@@ -13,6 +13,15 @@ export const doc = CheckBoxGroup => { | |||
.intrinsicElement('div'); | |||
|
|||
DocumentedCheckBoxGroup.propTypes = { | |||
children: PropTypes.func.description( |
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.
Co-authored-by: Eric Soderberg <eric.soderberg@hpe.com>
Thanks for your contribution! |
Signed-off-by: gurkiran_singh gurkiransinghk@gmail.com
What does this PR do?
CheckBox
andCheckBoxGroup
can accept children nowWhere should the reviewer start?
Components => CheckBox and CheckBoxGroup
What testing has been done on this PR?
Added story
Children
in bothCheckBox
andCheckBoxGroup
How should this be manually tested?
Using Storybook
Any background context you want to provide?
N/A
What are the relevant issues?
#5221
Screenshots (if appropriate)
N/A
Do the grommet docs need to be updated?
Yes, I already did that
Should this PR be mentioned in the release notes?
Is this change backwards compatible or is it a breaking change?
backwards compatiable