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

Add gutters to the header consistent with the grid #1144

Merged
merged 2 commits into from
Jan 17, 2019

Conversation

NickColley
Copy link
Contributor

@NickColley NickColley commented Jan 16, 2019

This allows the header to line up with the grid properly.

Screenshots

Desktop

Before

desktop-before

After

desktop-after

Above tablet

Before

above-tablet-before

After

above-tablet-after

Tablet

Before

tablet-before

After

tablet-after

To do

  • Test cross browser

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1144 January 16, 2019 17:17 Inactive
@NickColley
Copy link
Contributor Author

Internet Explorer 8 is rendering oddly but I don't think it's related to this change, I'll try and fix that first so we can know that doing this work doesn't break Internet Explorer 8...

#1145

@edwardhorsford
Copy link
Contributor

🎉

@36degrees
Copy link
Contributor

This makes sense to me, but it'd be good if @dashouse could take a look as well.

Copy link

@dashouse dashouse left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@36degrees
Copy link
Contributor

36degrees commented Jan 17, 2019

(Needs a changelog entry)

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1144 January 17, 2019 11:37 Inactive
@36degrees
Copy link
Contributor

🚢

@NickColley NickColley self-assigned this Jan 17, 2019
This allows the header to line up with the grid properly.
@NickColley
Copy link
Contributor Author

I'm happy this works well in IE8 now that the other bug has been fixed.

@NickColley NickColley merged commit 88df9f8 into master Jan 17, 2019
@NickColley NickColley deleted the adjust-spacing-for-header-to-match-grid branch January 17, 2019 17:02
@NickColley NickColley added this to the [NEXT] milestone Jan 21, 2019
This was referenced Jan 31, 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.

5 participants