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

Remove bottom margin from navigation on tablets #2447

Merged
merged 2 commits into from
Dec 1, 2021
Merged

Conversation

36degrees
Copy link
Contributor

Split out from #2427.

The intention seems to be to add the 10px margin only when the navigation is displayed inline (desktop), so switch to using a media query instead of using the responsive margin mixin.

(Normally govuk-responsive-margin(2, "bottom") would create a 10px bottom margin on both mobile and tablet/desktop, except the margin on mobile is currently overridden by margin: 0)

Breakpoint Before After
Mobile (no change) mobile-before mobile-after
Tablet tablet-before tablet-after
Desktop (no change) desktop-before desktop-after

The intention seems to be to add the 10px margin only when the navigation is displayed inline (desktop), so switch to using a media query instead of using the responsive margin mixin.

(Normally `govuk-responsive-margin(2, "bottom")` would create a 10px bottom margin on both mobile and tablet/desktop, except the margin on mobile is currently overridden by `margin: 0`)
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2447 November 29, 2021 14:25 Inactive
@36degrees 36degrees merged commit 54c6638 into main Dec 1, 2021
@36degrees 36degrees deleted the header-tablet-margin branch December 1, 2021 12:00
@vanitabarrett vanitabarrett mentioned this pull request Dec 15, 2021
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