Skip to content

fix: lay out docs text and install controls for phones - #44

Merged
NaNIsaac merged 2 commits into
mainfrom
fix/mobile-layout
Oct 24, 2026
Merged

NaNIsaac merged 2 commits into
mainfrom
fix/mobile-layout

Conversation

@NaNIsaac

@NaNIsaac NaNIsaac commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Tier: low

  • Phones (640 px and narrower): code and menu text are 14 px, installer tabs, copy button and workflow links reach 40 px, long inline code wraps inside table cells instead of widening the table, and prose, heading and list spacing follow a phone rhythm. Command reference at 360 px drops 21845 to 21009 px, configuration 5078 to 4657 px.
  • Desktop is unchanged: document heights at 1280 px are identical and screenshots at 1280 px are pixel-identical to main (the home page demo animates; two captures of main differ by the same area).
  • Changes are in site/.vitepress/theme (style.css, HeroInstall.vue, HomeWorkflows.vue). No content removed, no new dependency.

Document height (px)

npm ci and npx vitepress build in site/ on main at f301969 (after the command reference gained the GitHub scan options) and on this branch merged with it, served under /codigest/. Chrome through Playwright, is_mobile, DPR 2.

page 360 main 360 PR 412 main 412 PR 1280 main 1280 PR
/ 4976 4974 4639 4652 3137 3137
/ko/ 5046 5064 4728 4741 3216 3216
/getting-started 6384 6145 5908 5677 4781 4781
/commands 21845 21009 20032 19373 15944 15944
/configuration 5078 4657 4530 4226 3350 3350

The two home pages do not get shorter: raising the installer tabs, the copy button and the code to the 14 px and 40 px minimums adds about as much as the tighter spacing removes.

Text nodes under 14 px at 360 px: 58 to 2 on the home page, 433 to 3 on the command reference (the remaining ones are the hidden skip link and a description label). Controls under 40 px by layout box: home 25 to 15, commands 70 to 53, configuration 27 to 11; the rest are the drawer rows and inline links.

Tried and left out

The option tables on /commands break flags such as --no-copy across lines because the first column is about 80 px wide. Keeping each <code> on one line does not help: -c, --copy, --no-copy needs about 190 px, which leaves the description column about 110 px, and long codes such as <user_instruction> then push the table wider than the screen. The PR instead lets long inline code wrap inside table cells. The workflow cards on the home page stay one per row because the commands do not fit 160 px.

Screenshots (main on the left, this branch on the right): docs/mobile/shots/codigest/ in the NaNIsaac workspace. Images could not be uploaded to the PR from the CLI.

Checks: conventions.py check 0 findings; npx vitepress build succeeds.

@NaNIsaac
NaNIsaac merged commit 6027311 into main Oct 24, 2026
5 checks passed
@NaNIsaac
NaNIsaac deleted the fix/mobile-layout branch October 24, 2026 08:09
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