Skip to content

Fix docs container gutters across mid-width breakpoints#199

Merged
diogo-filipe-costa merged 2 commits intomainfrom
codex/docs-container-gutters-fix
Apr 9, 2026
Merged

Fix docs container gutters across mid-width breakpoints#199
diogo-filipe-costa merged 2 commits intomainfrom
codex/docs-container-gutters-fix

Conversation

@diogo-filipe-costa
Copy link
Copy Markdown
Contributor

@diogo-filipe-costa diogo-filipe-costa commented Apr 9, 2026

What was found

The docs page container lost its side gutters across mid-width viewports, so the content could sit flush against the viewport edges.

Why it happened

The docs site increased the container max width to 1280px, but the gutter handoff logic still matched the older container behavior. That mismatch stopped the fixed gutters too early.

How it was fixed

Use the responsive horizontal margin helper on the docs container and keep margin: auto only once the 1280px container plus gutters fully fits. This restores consistent gutters across the affected range.

How it was vs how it was fixed:
comparison-1200

@diogo-filipe-costa diogo-filipe-costa self-assigned this Apr 9, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for ofh-design-system-storybook canceled.

Name Link
🔨 Latest commit f8a8064
🔍 Latest deploy log https://app.netlify.com/projects/ofh-design-system-storybook/deploys/69d7c63e1d346200087de31b

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for ofh-design-system-docs ready!

Name Link
🔨 Latest commit f8a8064
🔍 Latest deploy log https://app.netlify.com/projects/ofh-design-system-docs/deploys/69d7c63eceb40500089e97bd
😎 Deploy Preview https://deploy-preview-199--ofh-design-system-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@diogo-filipe-costa diogo-filipe-costa marked this pull request as ready for review April 9, 2026 15:32
@diogo-filipe-costa diogo-filipe-costa changed the title Codex/docs container gutters fix Fix docs container gutters across mid-width breakpoints Apr 9, 2026
Copy link
Copy Markdown
Contributor

@cfgreene cfgreene left a comment

Choose a reason for hiding this comment

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

All good

@diogo-filipe-costa diogo-filipe-costa merged commit 6598885 into main Apr 9, 2026
9 checks passed
@diogo-filipe-costa diogo-filipe-costa deleted the codex/docs-container-gutters-fix branch April 9, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants