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

Revert spacing changes to service manual homepage #3570

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

MartinJJones
Copy link
Contributor

@MartinJJones MartinJJones commented Feb 24, 2025

What

Revert spacing changes to service manual homepage

Why

The govuk-main-wrapper class adds the correct spacing to the view templates and follows the design system guidance.

When used on the service-manual and service-toolkit homepage, it adds extra unwanted spacing at the top of the blue banner. To fix this, the new exclude_main_wrapper_class method has been used to remove the govuk-main-wrapper class from these pages.

Visual Changes

Before

Service Manual

Before After
Screenshot 2025-02-24 at 11 41 46 Screenshot 2025-02-24 at 11 45 23

Service Toolkit

Before After
Screenshot 2025-02-24 at 11 42 11 Screenshot 2025-02-24 at 11 44 48

The `govuk-main-wrapper` class adds the correct spacing to the view templates and follows the design system guidance.

When used on the service-manual and service-toolkit homepage, it adds extra unwanted spacing at the top of the blue banner. To fix this, the new `exclude_main_wrapper_class` method has been used to remove the `govuk-main-wrapper` class from these pages.
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-3570 February 24, 2025 11:41 Inactive
@MartinJJones MartinJJones marked this pull request as ready for review February 24, 2025 11:47
Copy link
Contributor

@JamesCGDS JamesCGDS left a comment

Choose a reason for hiding this comment

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

LGTM, good spot 👍

@MartinJJones MartinJJones merged commit 5f635e7 into main Feb 24, 2025
12 checks passed
@MartinJJones MartinJJones deleted the revert-service-manual-spacing-changes branch February 24, 2025 15:21
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