Skip to content

Fix content showing in sidebar region issue #1160#1161

Closed
trubach wants to merge 1 commit into5.xfrom
sidebar-content-fix-1160
Closed

Fix content showing in sidebar region issue #1160#1161
trubach wants to merge 1 commit into5.xfrom
sidebar-content-fix-1160

Conversation

@trubach
Copy link
Copy Markdown
Contributor

@trubach trubach commented Feb 20, 2026

📝 Description

Fixes #1160

The issue was that if the sidebar area was blank, no html structure was generated, and so the first content the ilw-columns tag would encounter would be the actual content, which meant it would show the content in the sidebar region.

To solve the issue, I added a <div> tag around the sidebar area, which forces the sidebar to render and puts the content in the proper place.


✅ 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)

@trubach trubach added the 5.x Work that needs to be done on the 5.x branch label Feb 20, 2026
@trubach
Copy link
Copy Markdown
Contributor Author

trubach commented Feb 24, 2026

I'm closing this PR in favor of PR #1165 - It addresses the issue as well, but fixes a number of other issues

@trubach trubach closed this Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5.x Work that needs to be done on the 5.x branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Page content showing in sidebar region

1 participant