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
fix: TileGroup to accept wrappers around RadioTiles. #16001
fix: TileGroup to accept wrappers around RadioTiles. #16001
Conversation
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Tested locally with both divs and grid wrappers and this works as expected. 🙌
@Gururajj77 feel free in the future to create a test story (that can be removed before merge) for easier review/testing, especially on PRs that need a design review.
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.
LGTM 👍 ✅
dccc214
Sure, will do this from next. |
…ystem#16001) * fix: fixes the default withOverlay prop * fix: fixed the tilegroup to accept wrappers around the radiotile * fix: removed comments --------- Co-authored-by: Taylor Jones <tay1orjones@users.noreply.github.com> Co-authored-by: Andrea N. Cardona <cardona.n.andrea@gmail.com> Co-authored-by: TJ Egan <tw15egan@gmail.com>
Closes #15884
Fixes the TileGroup to accept wrappers around RadioTiles when used under TileGroup
Changelog
New
Changed
Testing / Reviewing
Please add wrappers around RadioTiles when using under TileGroup