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

Fix hints for disabled checkboxes or radios appearing darker than the associated labels #2615

Merged
merged 3 commits into from
May 13, 2022

Conversation

36degrees
Copy link
Contributor

@36degrees 36degrees commented May 12, 2022

When checkboxes or radios with hints are disabled, the hint text ends up being darker than the label because the opacity of the label is reduced to 50%, but the opacity of the hint is not changed and so remains a darker grey.

Reduce the opacity of the hints associated with disabled radios and checkboxes so that it is also ‘greyed out’ proportional to the label.

Update the examples for disabled radios and checkboxes to include hints so that we have a visual example of this in the review app.

Before

Screenshot 2022-05-12 at 17 11 55

Screenshot 2022-05-12 at 17 11 49

After

Screenshot 2022-05-12 at 17 12 10

Screenshot 2022-05-12 at 17 12 08

Fixes #2581.

It’s been flagged that when checkboxes or radios with hints are disabled, the hint text ends up being darker than the label because the opacity of the label is changed but the opacity of the hint is not.

Update the examples for disabled radios and checkboxes to include hints so that we have a visual example of this in the review app.
When checkboxes or radios with hints are disabled, the hint text ends up being darker than the label because the opacity of the label is reduced to 50%, but the opacity of the hint is not changed and so remains a darker grey.

Reduce the opacity of the hints associated with disabled radios and checkboxes so that it is also ‘greyed out’ proportional to the label.
@36degrees 36degrees changed the title Reduce hint opacity for disabled checkboxes/radios Fix hints for disabled checkboxes/radios appearing as darker than the associated labels May 12, 2022
@36degrees 36degrees changed the title Fix hints for disabled checkboxes/radios appearing as darker than the associated labels Fix hints for disabled checkboxes/radios appearing darker than the associated labels May 12, 2022
Copy link
Member

@querkmachine querkmachine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all makes sense to me 👍

@36degrees 36degrees requested a review from Ciandelle May 13, 2022 09:05
Copy link

@Ciandelle Ciandelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy with the change

@36degrees 36degrees merged commit 3cdea58 into main May 13, 2022
@36degrees 36degrees deleted the disabled-hint-color branch May 13, 2022 10:44
@EoinShaughnessy EoinShaughnessy changed the title Fix hints for disabled checkboxes/radios appearing darker than the associated labels Fix hints for disabled checkboxes or radios appearing darker than the associated labels May 16, 2022
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 this pull request may close these issues.

Hints on disabled checkboxes and radios are styled darker than the label
3 participants