Skip to content

refactor(page-content): use the theme's layout-boundary-inline for the prose column - #2380

Merged
isaque-bock-azion merged 1 commit into
release/new-azion-docsfrom
refactor/page-content-layout-boundary
Sep 21, 2026
Merged

isaque-bock-azion merged 1 commit into
release/new-azion-docsfrom
refactor/page-content-layout-boundary

Conversation

@isaque-bock-azion

Copy link
Copy Markdown
Contributor

What

The article body in PageContent.astro now takes the theme's own layout-boundary-inline utility instead of the local px-container, aligning it with the layout primitives DocsHomeLayout and DocsDirectoryLayout already use.

The three narrative comment blocks describing the previous masthead/pagination arrangement were removed.

Note for review

px-container was a local @utility that set max-width: var(--container-4xl) + auto margins and a gutter; layout-boundary-inline (from @aziontech/theme) sets padding-inline: var(--layout-boundary-inline) only. Used alone, the prose section therefore loses its measure cap — DocProse itself is w-full. The other two layouts pair the utility with mx-auto w-full max-w-(--layout-measure-site).

Worth confirming on a wide viewport that this is the intended width, and whether DocPagination (still on px-container) and the .doc-page-banner padding math (still keyed to --container-4xl) should follow.

…e prose column

Swap `px-container` for the theme's own `layout-boundary-inline` utility on the
article body, matching the layout primitives the docs layouts already use, and drop
the three narrative comment blocks that explained the previous arrangement.
@isaque-bock-azion
isaque-bock-azion requested a review from a team as a code owner September 21, 2026 20:21
@isaque-bock-azion
isaque-bock-azion merged commit 2dc07b3 into release/new-azion-docs Sep 21, 2026
6 of 8 checks passed
@isaque-bock-azion
isaque-bock-azion deleted the refactor/page-content-layout-boundary branch September 21, 2026 20:32
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