Skip to content

docs(ui): body copy off-center on pages without a contents rail - #1331

Merged
alandefreitas merged 1 commit into
cppalliance:developfrom
alandefreitas:docs/toc-margin
Sep 30, 2026
Merged

alandefreitas merged 1 commit into
cppalliance:developfrom
alandefreitas:docs/toc-margin

Conversation

@alandefreitas

Copy link
Copy Markdown
Collaborator

On pages with no section headings, the "On this page" rail is dropped, and the article column grows wider than the reading measure. The title and the links at the bottom then move to the center while the paragraphs and lists stay on the left edge. This change makes every block follow the same centering rule. Pages that keep the rail look the same as before.

Changes

Four stylesheet rules that set block margins now touch only the top and bottom margins, so the side margins are left to the rule that centers each block on the reading measure.

Testing

I rendered the docs locally and compared a page without a rail (the Extensions overview) against one with a rail (Handlebars Extensions).

Documentation

None needed.

@github-actions

github-actions Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

🧾 Changes by Scope

Scope Lines Δ% Lines Δ Lines + Lines - Files Δ Files + Files ~ Files ↔ Files -
📄 Docs 100% 14 10 4 1 - 1 - -
Total 100% 14 10 4 1 - 1 - -

Legend: Files + (added), Files ~ (modified), Files ↔ (renamed), Files - (removed)

🔝 Top Files

  • docs/ui/src/css/doc.css (Docs): 14 lines Δ (+10 / -4)

Generated by 🚫 dangerJS against cc7b743

@codecov

codecov Bot commented Sep 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.12%. Comparing base (6a984f6) to head (cc7b743).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1331      +/-   ##
===========================================
+ Coverage    79.15%   83.12%   +3.96%     
===========================================
  Files          384       35     -349     
  Lines        39441     3662   -35779     
  Branches      7848      844    -7004     
===========================================
- Hits         31221     3044   -28177     
+ Misses        5362      410    -4952     
+ Partials      2858      208    -2650     
Flag Coverage Δ
bootstrap 83.12% <ø> (ø)
cpp ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cppalliance-bot

cppalliance-bot commented Sep 30, 2026 •

Copy link
Copy Markdown

An automated preview of the documentation is available at https://1331.mrdocs.prtest2.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-09-30 21:10:23 UTC

@alandefreitas
alandefreitas added this pull request to the merge queue Sep 30, 2026
Merged via the queue into cppalliance:develop with commit 917352d Sep 30, 2026
19 checks passed
@alandefreitas alandefreitas changed the title docs(ui): body copy off-centre on pages without a contents rail docs(ui): body copy off-center on pages without a contents rail Oct 1, 2026
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.

2 participants