-
Notifications
You must be signed in to change notification settings - Fork 9
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
RadioGroup - make legend optional #388
Labels
🎨 design issue
Task is for designers
⌨️ dev issue
Task is for developers
🎓 junior issue
Good for juniors
🚨 new::bug
Something isn't working
Milestone
Comments
Continuing with the ticket Radio Group - add global caption/hint improvements #402 |
Dev is done |
We looked at the component and the padding was moved to the legend in the code, but we could not verify if everything is done, as design is not updated yet. |
Dev is done |
After merge this task can be closed as done. But we will wait for the tokens & git ignore topic to be resolved before doing it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
🎨 design issue
Task is for designers
⌨️ dev issue
Task is for developers
🎓 junior issue
Good for juniors
🚨 new::bug
Something isn't working
In Figma the legend isn't optional yet. The developed component already has an optional legend.
The current layout must be changed, because if the legend is hidden, there remains an undesired top padding on the RadioStack.
Design tasks:
paddingTop: "Forms.RadioGroup.SM.RadioStack.PaddingTop";
Dev tasks:
The text was updated successfully, but these errors were encountered: