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

Align phase banner on service manual homepage #3296

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

MartinJJones
Copy link
Contributor

@MartinJJones MartinJJones commented Aug 9, 2024

What

  • Align phase banner on service manual homepage
  • Add new service_manual_homepage? presenter method

Why

When on the service_manual_homepage, the phase banner will be wrapped in a div element with the govuk-width-container class to align it with the rest of the content on the page.

The service manual homepage layout is different from the rest of the service manual as it contains an app-hero banner that needs to span the full width of the page with a blue background, but we still want the phase banner to be contained.

Visual changes

Service manual homepage

Before After
Screenshot 2024-08-09 at 13 56 22 Screenshot 2024-08-09 at 13 56 58

Add new `service_manual_homepage?` presenter method.

When on the service_manual_homepage, the phase banner will be wrapped in a div element with the `govuk-width-container` class to align it with the rest of the content on the page.

The service manual homepage layout is different from the rest of the service manual as it contains an app-hero banner that needs to span the full width of the page with a blue background, but we still want the phase banner to be contained.
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-3296 August 9, 2024 12:55 Inactive
Copy link
Contributor

@CodeSonia CodeSonia left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this 🌟

@MartinJJones MartinJJones merged commit 82fcced into main Aug 9, 2024
12 checks passed
@MartinJJones MartinJJones deleted the phase-banner-service-manual-homepage branch August 9, 2024 13:03
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