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 the govuk-width-container CSS class from recruitment banner #44

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

MartinJJones
Copy link
Contributor

@MartinJJones MartinJJones commented Jan 14, 2025

What

Remove the govuk-width-container CSS class from recruitment banner

Why

The govuk-width-container is not required, the layout classes from the design system can be added in each application where the govuk_web_banner is used.

This will help improve the spacing on mobile and avoid extra spacing to the left and right of the component.

https://design-system.service.gov.uk/styles/layout/#common-layouts

Visual Changes

Page: https://www.gov.uk/government/collections/tax-compliance-detailed-information

Before After
banner-before banner-after

The `govuk-width-container` is not required, the layout classes from the design system can be added in each application where the govuk_web_banner is used.

This will help improve the spacing on mobile and avoid extra spacing to the left and right of the component.
@MartinJJones MartinJJones force-pushed the remove-govuk-width-container branch from 5c38f97 to 01ee6b2 Compare January 14, 2025 12:16
@MartinJJones MartinJJones marked this pull request as ready for review January 14, 2025 12:16
@MartinJJones MartinJJones changed the title Remove the govuk-width-container from recruitment banner Remove the govuk-width-container CSS class from recruitment banner Jan 14, 2025
Copy link
Collaborator

@KludgeKML KludgeKML left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@MartinJJones MartinJJones merged commit 1862089 into main Jan 14, 2025
10 checks passed
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.

2 participants