Skip to content

fix(website): prevent iOS Safari text inflation in docs tables - #665

Merged
macalbert merged 1 commit into
mainfrom
macalbert/ios-safari-text-inflation-7d9545
Nov 6, 2026
Merged

macalbert merged 1 commit into
mainfrom
macalbert/ios-safari-text-inflation-7d9545

Conversation

@macalbert

@macalbert macalbert commented Nov 6, 2026 •

Copy link
Copy Markdown
Owner

Pull Request

What does this PR do?

Fixes oversized text in docs tables on iOS Safari. On narrow viewports, .docs-table becomes a horizontally scrollable block with white-space: nowrap on every cell, so the table is far wider than the viewport. iOS Safari then applies text inflation and renders the description column at a much larger font size, which pushes most of the text out of view.

  • Set -webkit-text-size-adjust: 100% / text-size-adjust: 100% on html so Safari keeps the authored font sizes.
  • Limit white-space: nowrap to the first column (codes, flags) so description cells wrap instead of overflowing.

Related issues

None.

Type of change

  • Feature
  • Bugfix
  • Refactor
  • Documentation
  • Other

Checklist

  • Tests added/updated (if needed)
  • Docs updated (if needed)
  • Lint/format pass

Notes for reviewer

CSS only. pnpm build in src/website passes. Text inflation only happens on real iOS Safari, so the final check needs an iPhone (for example the exit codes table on /ca/docs/cli/exec/).

Summary by CodeRabbit

  • Bug Fixes
    • Improved documentation table readability on mobile by keeping first-column text on one line while allowing tables to scroll horizontally.
    • Standardized text sizing in Safari, including iOS, for more consistent page rendering.

@github-actions github-actions Bot added bug Something isn't working size/S Small change — 10-49 lines / 2-4 files labels Nov 6, 2026
@coderabbitai

coderabbitai Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The documentation styles now set standard and WebKit text-size adjustment to 100%. The mobile table rule ends before the first-cell selector, keeping white-space: nowrap scoped to the first table cell.

Changes: Documentation page styling

Layer / File(s) Summary
Responsive documentation styles
src/website/src/components/DocsArticle.astro, src/website/src/styles/global.css
The mobile .docs-table rule ends before the first-cell selector. The html rule sets standard and WebKit text-size adjustment to 100%.

Estimated code review effort: 1 (Trivial) | ~4 minutes

Merge Risk: Minimal · up to df02c

No merge-blocking issue was identified. The changes are intended to improve table readability on narrow iOS screens.

Pre-merge checks: Passed 7

Check name Status Explanation
Title check Passed The title describes the main change and meets the Conventional Commits requirements in .commitlintrc.json.
Description check Passed The description covers the required sections and explains the change. The checklist is unchecked, and iPhone verification remains pending.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Architecture Boundaries Passed The PR changes only website files. It does not change files under src/envilder/core/domain/ or src/envilder/core/application/.
Tests Accompany Code Changes Passed The PR changes only files under src/website/. It changes no source files under src/envilder/ or src/sdks/.
Release Coherence Passed The PR changes only two website files. No specified version field changed.
Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 7c4d99b and df02c74.

Files selected for processing: 2

  • src/website/src/components/DocsArticle.astro
  • src/website/src/styles/global.css

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

@macalbert macalbert self-assigned this Nov 6, 2026
@macalbert
macalbert merged commit 67582b7 into main Nov 6, 2026
30 of 33 checks passed
@macalbert
macalbert deleted the macalbert/ios-safari-text-inflation-7d9545 branch November 6, 2026 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size/S Small change — 10-49 lines / 2-4 files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant