Skip to content

Updated .usa-alert--info-important style to bold text.#7258

Merged
jmax-gsa merged 1 commit intomainfrom
LG-7970-bolden-irs-reproofing-text
Nov 1, 2022
Merged

Updated .usa-alert--info-important style to bold text.#7258
jmax-gsa merged 1 commit intomainfrom
LG-7970-bolden-irs-reproofing-text

Conversation

@jmax-gsa
Copy link
Contributor

@jmax-gsa jmax-gsa commented Oct 31, 2022

🎫 Ticket

[https://cm-jira.usa.gov/browse/LG-7970]

🛠 Summary of changes

Set font-weight style on the style to 500, per slack conversation

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Bring up the proofing welcome screen for IRS re-proofing, and examine the text

👀 Screenshots

LG-7970

@jmax-gsa jmax-gsa self-assigned this Oct 31, 2022
@jmax-gsa jmax-gsa force-pushed the LG-7970-bolden-irs-reproofing-text branch from a128fc0 to 07bd7ab Compare November 1, 2022 14:07
@jmax-gsa jmax-gsa marked this pull request as ready for review November 1, 2022 14:51
Copy link

@benjaminchait benjaminchait left a comment

Choose a reason for hiding this comment

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

+1 lgtm

@jmax-gsa jmax-gsa merged commit dc8880e into main Nov 1, 2022
@jmax-gsa jmax-gsa deleted the LG-7970-bolden-irs-reproofing-text branch November 1, 2022 15:02
.usa-alert--info-important {
@include u-bg('accent-cool-darker');
color: color('white');
font-weight: 500;
Copy link
Contributor

Choose a reason for hiding this comment

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

Because we only define 400 and 700 font weights, and due to how fallback weight behavior would choose the normal font weight, I wouldn't expect this change to have any effect, and in my testing it appears visually the same as if this line didn't exist at all.

If we wanted bold, I would expect...

Suggested change
font-weight: 500;
font-weight: bold;

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.

3 participants