-
Notifications
You must be signed in to change notification settings - Fork 320
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
Removing font style adjustments #1441
Conversation
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.
The header in legacy mode seems quite a lot bigger, is this intentional?
Apart from that I've looked at the before and after with legacy mode and it all looks good to me.
Would be good to get @dashouse 's sign off.
left: 0; | ||
|
||
margin: auto; | ||
} | ||
} | ||
|
||
// Begin adjustments for font baseline offset | ||
// These should be removed when the font is updated with the correct baseline | ||
@if $govuk-use-legacy-font { |
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.
I like using this instead of compatibility mode, feels more explicit 👍
Looks good to me 👏 |
padding-left: 8px; | ||
} @else { | ||
@include govuk-font($size: 16, $weight: bold, $line-height: 1); | ||
padding-top: 5px; |
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.
Non-blocking this could be short hand as a one liner.
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.
Excellent work both 👏
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.
Needs a CHANGELOG entry
672cf78
to
6a42186
Compare
6a42186
to
bcf65db
Compare
Based off @dashouse initial spike #1356 and linked to #1434.
This PR moves the adjustments behind a sass variable called
$govuk-use-legacy-font
. This should mean that services not running in compatibility mode should have a slightly smaller compiled CSS without the adjustments.Components checked
Components affected and updated