Skip to content

Refactor of the page.html.twig template, eliminate content page template#1165

Closed
trubach wants to merge 3 commits into5.xfrom
sidebar-template-logic
Closed

Refactor of the page.html.twig template, eliminate content page template#1165
trubach wants to merge 3 commits into5.xfrom
sidebar-template-logic

Conversation

@trubach
Copy link
Contributor

@trubach trubach commented Feb 23, 2026

📝 Description

This refactor takes the approach where the paragraphs that need to be outside of the two-column layout are removed from the page.content variable and put into a separate page.full_width_content variable.

Still to do:

  • The body field on full-width pages goes to the width of the browser. We need to figure out a way to make the body field default to a normal page width without affecting other items
  • On the news content type, the sub-title and author info are the width of the browser

Fixes #1102


✅ Peer Review Checklist

Reviewers: Please verify the following before approving.

  • The pull request links to the issue it is addressing in the comment and in the "Development" section of the PR
  • There is a short summary that describes how the fix is implemented
  • Verifying work: Verify that the changes made are addressing the linked issue
  • Accessibility: Changes that might impact accessibility are reviewed to work with keyboard navigation and screen readers
  • Responsive: Layout is tested on desktop, tablet, and mobile screens
  • Config changes: Any configuration updates are tested and verified using the Distribution Update import
  • Security: Output is sanitized (using t(), Twig auto-escaping, etc.). No secrects included (API keys, etc.)
  • Cleanliness: All debug code (ksm(), dpm(), console.log) has been removed.
  • Comments: Complex logic is explained inline.
  • Documentation: Any relevant documentation has been updated (this can be a separate issue if needed)

📸 Screenshots / Video (Optional)

If this is a UI change, please attach a screenshot or a quick screen recording of the change in action.

@trubach
Copy link
Contributor Author

trubach commented Feb 26, 2026

Abandoning this PR / branch. See PR #1170 for the hopefully final PR

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.

1 participant