Skip to content
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

Native RadioTile required validation not supported #972

Closed
brunnerh opened this issue Jan 5, 2022 · 1 comment
Closed

Native RadioTile required validation not supported #972

brunnerh opened this issue Jan 5, 2022 · 1 comment

Comments

@brunnerh
Copy link
Contributor

brunnerh commented Jan 5, 2022

Currently a selection of a radio tile cannot be enforced using native form validation. To use this the same name has to be set on all radio inputs along with the required attribute.

Suggested changes:

  • name should be propagated from TileGroup to RadioTile. It makes no sense for the elements of the group to have different names; they would no longer be part of the same group. The name property would then be removed from RadioTile.
  • required should also be propagated down.

I implemented the logic, but currently there is still an upstream layout issue. (The validation message is displayed in the wrong place.)

@metonym metonym added the flagship Question for @carbon-design-system/carbon label Jan 10, 2022
@metonym metonym removed the flagship Question for @carbon-design-system/carbon label Dec 11, 2022
@brunnerh
Copy link
Contributor Author

brunnerh commented Oct 3, 2023

@brunnerh brunnerh closed this as completed Oct 3, 2023
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 a pull request may close this issue.

2 participants