-
Notifications
You must be signed in to change notification settings - Fork 102
Ix2377 radio #1928
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Ix2377 radio #1928
Conversation
馃 Changeset detectedLatest commit: 40f788a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
We added a margin around the button but this area is still not interactive.
Component test is missing.
Hi Lukas, |
|
|
馃挕 What is the current behavior?
Radio's visible size is correct, but the active area needs to be larger to meet WCAG guidelines
When using a radio without label, the right space remains
GitHub Issue Number: # GitHub issue doesn't exit
jira issue number IX-2377
馃啎 What is the new behavior?
Active area of the radio without label is 24x24
visible size of radio is 20x20
Radio with label: the space between Checkbox and label is 8px/0.5rem
馃弫 Checklist
A pull request can only be merged if all of these conditions are met (where applicable):
pnpm test
)pnpm lint
)pnpm build
, changes pushed)馃懆鈥嶐煉 Help & support