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

Improve print styles for layout-footer component #4178

Merged
merged 2 commits into from
Sep 2, 2024

Conversation

matthillco
Copy link
Contributor

@matthillco matthillco commented Aug 28, 2024

What

This improves the print styles for the layout-footer component as part of the work to improve print styles for page layouts. Trello.

  • Extra borders and padding are removed
  • internal elements now run flush to their left/right edges

Note that for consistency all variants of the layout-footer now render identically when printed .

Why

Although we made improvements to the layout-footer as part of #4073, we did not look at how the component might interact with other components on the page. This PR further improves the component in context of page types.

Visual Changes

Before and After: Component on a typical page

image

@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4178 August 28, 2024 15:28 Inactive
@matthillco matthillco force-pushed the print-styles-layout-footer branch from 9a1587b to 2c2fcc0 Compare August 28, 2024 16:26
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4178 August 28, 2024 16:27 Inactive
@matthillco matthillco force-pushed the print-styles-layout-footer branch from 2c2fcc0 to e646455 Compare August 30, 2024 15:16
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4178 August 30, 2024 15:16 Inactive
@matthillco matthillco force-pushed the print-styles-layout-footer branch from e646455 to 7dccf80 Compare August 30, 2024 15:24
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4178 August 30, 2024 15:24 Inactive
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4178 August 30, 2024 16:11 Inactive
@matthillco matthillco changed the title [WIP] Improve print styles for layout-footer component Improve print styles for layout-footer component Sep 2, 2024
Copy link
Contributor

@AshGDS AshGDS left a comment

Choose a reason for hiding this comment

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

LGTM so will approve, just had a couple questions which may result in changes.

@matthillco matthillco force-pushed the print-styles-layout-footer branch from a46fcac to 635c9e9 Compare September 2, 2024 15:32
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4178 September 2, 2024 15:32 Inactive
@matthillco matthillco merged commit 6f6a821 into main Sep 2, 2024
12 checks passed
@matthillco matthillco deleted the print-styles-layout-footer branch September 2, 2024 16:09
@matthillco matthillco mentioned this pull request Sep 3, 2024
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