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

Explicitly set font weight on warning-text component #1585

Merged
merged 2 commits into from
Sep 23, 2019

Conversation

36degrees
Copy link
Contributor

Explicitly set the font-weight of the warning text component, and move the font styles to the __text element (the __icon element has its own size, weight and colour)

Warning text is otherwise only bold because of the user-agent styling of <strong> elements. When used with GOV.UK Elements, which includes a reset stylesheet, the font-weight is reset to normal, which means the warning text is currently set in a normal weight.

Before

Legacy mode OFF

WT Before

Legacy mode ON

WT Legacy Before

After

Legacy mode OFF

WT After

Legacy mode ON

WT Legacy after

Explicitly set the font-weight of the warning text component, and move the font styles to the __text element (the __icon element has its own size, weight and colour)

Warning text is otherwise only bold because of the user-agent styling of <strong> elements. When used with GOV.UK Elements, which includes a reset stylesheet, the font-weight is reset to normal, which means the warning text is currently set in a normal weight.
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1585 September 23, 2019 09:15 Inactive
Copy link
Contributor

@NickColley NickColley left a comment

Choose a reason for hiding this comment

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

Needs a CHANGELOG entry but this is a nice catch 👍

@NickColley NickColley added this to the Next milestone Sep 23, 2019
@36degrees 36degrees changed the title Explicitly set font weight of warning-text Explicitly set font weight on warning-text component Sep 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants