Skip to content

docs: Scroll the schema table in place on phones - #4388

Merged
ntucker merged 1 commit into
masterfrom
claude/project-thread-0bgxwd
Oct 14, 2026
Merged

ntucker merged 1 commit into
masterfrom
claude/project-thread-0bgxwd

Conversation

@ntucker

@ntucker ntucker commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

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-scroll wrapper in theme/MDXComponents. This wraps them in .table-scroll directly: 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-scroll wrapping 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.

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
@changeset-bot

changeset-bot Bot commented Oct 14, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 93fc419

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-14T21:25:36.768554Z 93fc419 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

Copy link
Copy Markdown
Contributor

Docs preview: https://docs-site-pr-4388-data-client.vercel.app

Built from 93fc419 merged into master (this build).

@ntucker
ntucker added this pull request to the merge queue Oct 14, 2026
Merged via the queue into master with commit 50e268f Oct 14, 2026
28 checks passed
@ntucker
ntucker deleted the claude/project-thread-0bgxwd branch October 14, 2026 22:24

This branch was successfully deployed

1 active deployment
docs-preview — 93fc4193 Deployed Oct 14, 2026 by ntucker via docs deploy #3237
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