Repository navigation
docs: Scroll the schema table in place on phones - #4388
Conversation
The Schema table (normalization, overview, schema API) and the Endpoint Conditions table were raw JSX <table>s, which MDX doesn't pass through the table-scroll wrapper in theme/MDXComponents. On phones they widened the page to ~600px so the whole page zoomed out. Wrap them in .table-scroll directly (English and translations; translation locks re-finalized). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D5nL5BkMxiNDSNW9PqLavJ
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Docs preview: https://docs-site-pr-4388-data-client.vercel.app Built from 93fc419 merged into |
Requested by Nathaniel · project thread
Before: on phones, the schema table on Normalization, Overview and the Schema API page widened the page to about 600px, so the whole page zoomed out (measured 600px scroll width at a 320px viewport).
After: the page stays at the viewport width (320, 375 and 414px, light and dark, React and Vue docs) and the table scrolls sideways inside its own box.
The tables were hand-written JSX
<table>s, which MDX doesn't route through the.table-scrollwrapper intheme/MDXComponents. This wraps them in.table-scrolldirectly: the schema table partial and the Endpoint Conditions table on Overview, in English and the es/pt-BR/zh-Hans translations (translation locks re-finalized; the translations were current before).Website preview: https://docs-site-pr-4388-data-client.vercel.app/docs/concepts/normalization
🤖 Generated with Claude Code
https://claude.ai/code/session_01D5nL5BkMxiNDSNW9PqLavJ
Note
Low Risk
Documentation-only markup and i18n lock updates; no runtime or API changes.
Overview
Fixes mobile docs layout where hand-written JSX
<table>blocks (schema table partial and Endpoint Conditions on Overview) bypassed the theme’s automatic.table-scrollwrapping and forced the page wider than the viewport.Those tables are now wrapped in
<div className="table-scroll">in English and mirrored in es, pt-BR, and zh-Hans; translation lock hashes for the touched doc paths are updated accordingly.Reviewed by Cursor Bugbot for commit 93fc419. Bugbot is set up for automated code reviews on this repo. Configure here.