NDS-493 update radio button documentation #387
Conversation
docs/pages/components/radio.mdx
Outdated
| > | ||
| <Radio | ||
| hint="Some helpful hint text" | ||
| label="Yes" |
There was a problem hiding this comment.
Do we want to keep casing consistent across value attributes.
if we keep to lower casing, that keeps those consistent across pair react and html examples
e.g. value="yes" instead of value="Yes"
There was a problem hiding this comment.
Might need to check across examples and amend where needed.
dalecreativ
left a comment
dalecreativ
left a comment
There was a problem hiding this comment.
Just a few comments to action, mainly around code example blocks.
Give me nudge if needed
-Updated examples with unique form group names -changed value to lowercase
|
Because we've dound a potential issue in the FormGroup handling of name prop, we might need to revert back to the FormGroup carrying the name and remove name from nested controls. Sorry @shilpanice |
-Removed name attribute from individual radios
No worries Dale...ALl updated now!! Thank you |
https://nicedigital.atlassian.net/browse/NDS-493