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
USWDS - Radio Button - radio shows unchecked when disabled even when checked #4063
Conversation
…ked and disabled. added styles for the --tile version.
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.
Nice job! Just some minor comments
<input class="usa-radio__input usa-radio__input--tile" id="historical-marshall-2" type="radio" name="historical-figures" value="thurgood-marshall" disabled checked> | ||
<label class="usa-radio__label" for="historical-marshall-2">Thurgood Marshall</label> | ||
</div> | ||
|
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.
Fix indent and remove extra line on 24.
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.
@jeffffeffff there is still an extra line on line 24
Co-authored-by: James Mejia <s.jamesmejia@gmail.com>
@mejiaj Sorry, I dismissed your review, prematurely determining it was stale. Yet it was not! |
Closing in favor or more up-to-date PR. |
Fix for #3979
Description
When a radio input was disabled but also had the checked attribute there was no visual indicator of this combination of states. This PR adds styling to "fill-in" the radio input as if it were checked while also showing other styling indicating it's disabled state. Also adds examples on the radio input component of this :disabled:checked state, including an
.usa-radio__input--tile
variant example.