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

feat: Make border of Radio thicker when hovering #1696

Merged
merged 12 commits into from
Nov 20, 2024
Merged
Next Next commit
Increase radio stroke width on hover
VincentSmedinga committed Oct 23, 2024

Verified

This commit was signed with the committer’s verified signature.
not-an-aardvark Teddy Katz
commit 57ad9a91e143f786dea3edab573094b216f3de53
3 changes: 2 additions & 1 deletion packages/css/src/components/radio/radio.scss
Original file line number Diff line number Diff line change
@@ -39,7 +39,7 @@
.ams-radio__circle {
fill: none;
stroke: var(--ams-radio-circle-stroke);
stroke-width: 0.125rem;
stroke-width: var(--ams-radio-circle-stroke-width);
}

.ams-radio__checked-indicator {
@@ -54,6 +54,7 @@

.ams-radio__circle {
stroke: var(--ams-radio-circle-hover-stroke);
stroke-width: var(--ams-radio-circle-hover-stroke-width);
}

.ams-radio__checked-indicator {
4 changes: 3 additions & 1 deletion proprietary/tokens/src/components/ams/radio.tokens.json
Original file line number Diff line number Diff line change
@@ -33,6 +33,7 @@
},
"circle": {
"stroke": { "value": "{ams.color.primary-blue}" },
"stroke-width": { "value": "2px" },
"disabled": {
"stroke": { "value": "{ams.color.neutral-grey3}" }
},
@@ -43,7 +44,8 @@
}
},
"hover": {
"stroke": { "value": "{ams.color.dark-blue}" }
"stroke": { "value": "{ams.color.dark-blue}" },
"stroke-width": { "value": "3px" }
},
"invalid": {
"stroke": { "value": "{ams.color.primary-red}" },