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

New class to collapse columns for print #4224

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

matthillco
Copy link
Contributor

@matthillco matthillco commented Sep 13, 2024

What

Provide a new helper class to "collapse" row/column layouts when printing. This class forces all floated columns in the associated row to print at full page width. Trello

Why

When printing various page types, the 2/3rds + 1/3rds column layout causes issues with cropped content and a generally poor printing experience because the full page isn't used. By selectively collapsing floated columns, we can avoid most of these issues and improve the printing experience by using the full width of the page.

Before/After page type: GUIDE

Before After
image image

Before/After page type: DETAILED_GUIDE

Before After
image image

Before/After page type: HTML_PUBLICATION (Portrait, no change}

Before After
image image

Before/After page type: HTML_PUBLICATION (Landscape}

Before After
image image

@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4224 September 13, 2024 16:34 Inactive
@matthillco matthillco changed the title [WIP] New class for collapsing columns for print [WIP] New class to collapse columns for print Sep 16, 2024
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4224 September 17, 2024 09:38 Inactive
@matthillco matthillco marked this pull request as ready for review September 17, 2024 10:46
@matthillco matthillco changed the title [WIP] New class to collapse columns for print New class to collapse columns for print Sep 17, 2024
Copy link
Contributor

@andysellick andysellick left a comment

Choose a reason for hiding this comment

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

Looks good, only one question, also probably worth a CHANGELOG entry.

@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4224 September 18, 2024 11:36 Inactive
@matthillco matthillco merged commit eb2614f into main Sep 18, 2024
12 checks passed
@matthillco matthillco deleted the spike-collapse-grid-for-print branch September 18, 2024 11:44
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.

4 participants