Skip to content

ui: scroll wide tables inside their own box on narrow screens#64

Merged
acoshift merged 1 commit into
mainfrom
fix/mobile-table-overflow
Jul 2, 2026
Merged

ui: scroll wide tables inside their own box on narrow screens#64
acoshift merged 1 commit into
mainfrom
fix/mobile-table-overflow

Conversation

@acoshift

@acoshift acoshift commented Jul 2, 2026

Copy link
Copy Markdown
Member

What

Markdown tables wider than the viewport forced the whole page to scroll horizontally on mobile — on /networking/transform/ the document was 677px wide in a 390px viewport, cutting off the navbar and content.

A render-table.html markup hook now wraps every Markdown table in a .table-wrap scroll container: wide tables scroll inside their own box, the page never scrolls sideways. The border + radius move from the table to the wrapper, so the rounded frame stays put while the table scrolls under it. No content changes — the hook covers all Markdown tables (there are no raw-HTML tables in content/).

Before / after (390px, /networking/transform/ ops matrix)

before — page overflows, navbar cut after — table scrolls in its box after — scrolled right
before after scrolled

Desktop is unchanged (border/radius on the wrapper renders identically):

desktop

Verified

  • 390px viewport: document.scrollWidth 677 → 390 on /networking/transform/; /access/roles/ unaffected.
  • The wrapper scrolls to reveal the remaining columns (screenshot above).
  • Desktop /access/roles/ before/after screenshots pixel-compared — no visible change.

Markdown tables wider than the viewport (e.g. the transform ops matrix)
forced the whole page to scroll horizontally on mobile. A render-table
hook now wraps every table in a .table-wrap scroll container; the
border + radius move to the wrapper so the frame stays put while the
table scrolls under it. Desktop rendering is unchanged.
@deploys-app deploys-app Bot temporarily deployed to pr-64 July 2, 2026 22:18 Destroyed
@deploys-app

deploys-app Bot commented Jul 2, 2026

Copy link
Copy Markdown

Preview deleted (PR closed).

@acoshift acoshift merged commit b408732 into main Jul 2, 2026
1 check passed
@acoshift acoshift deleted the fix/mobile-table-overflow branch July 2, 2026 23:17
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