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

Update header alignment to match layout #165

Merged
merged 2 commits into from
Nov 21, 2019
Merged

Conversation

NickColley
Copy link
Contributor

@NickColley NickColley commented Nov 19, 2019

This pull request aims to address some spacing issues introduced when this format was updated to use the GOV.UK Design System by aligning the header and layout elements on the page.

It also expands the width of the area for the product name which stops it wrapping early, this is something we might want to consider pushing upstream to GOV.UK Frontend in the future.

Full size before Full size after
Smaller before Smaller after
Even smaller before Even smaller after

Fixes #164

@NickColley NickColley changed the title Update alignment Update header alignment to match layout Nov 19, 2019
@NickColley NickColley marked this pull request as ready for review November 19, 2019 16:52
@36degrees
Copy link
Contributor

It also expands the width of the area for the product name which stops it wrapping early, this is something we might want to consider pushing upstream to GOV.UK Frontend in the future.

Possibly related: alphagov/govuk-frontend#1502

Copy link

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

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

Nice one. I did a similar workaround on one application which makes me wonder if we'd need to make the header__logo container more flexible via govuk-frontend.

@NickColley
Copy link
Contributor Author

NickColley commented Nov 19, 2019

I've made a small tweak to align this a bit more:

Before After

@NickColley
Copy link
Contributor Author

@alex-ju if you could help us document where you've done similar things by commenting upstream here: alphagov/govuk-frontend#1502 that'd be really helpful.

@NickColley
Copy link
Contributor Author

NickColley commented Nov 19, 2019

I've asked the graphic designers if I could get an extra pair of eyes on this so will see what they think.

Latest:

@NickColley
Copy link
Contributor Author

Laurence B and Charlotte D from the GDS graphic designer community have said they approve of this change so I'm going to get it merged.

@NickColley NickColley merged commit 9c0c5ba into master Nov 21, 2019
@NickColley NickColley deleted the update-alignment branch November 21, 2019 15:31
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.

Wonky header
3 participants