Skip to content

fix(markdown): wrap long table cells without squeezing wide tables - #1056

Open
uvforce wants to merge 1 commit into
agegr:mainfrom
uvforce:fix/markdown-table-wrap
Open

uvforce wants to merge 1 commit into
agegr:mainfrom
uvforce:fix/markdown-table-wrap

Conversation

@uvforce

@uvforce uvforce commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

 .markdown-body th, .markdown-body td {
+  max-width: 240px;

Fixes #1052. Keep the natural table sizing and horizontal scrolling introduced by #650; bound long cell content rather than squeezing the entire table into the viewport.

Evidence

  • Before: The new production-mode browser regression fails because a two-column table stretches beyond the chat.
    After: The same browser suite passes all 12 checks at 1280px and 390px. Long headings, English/Chinese sentences, URLs, and inline code wrap; an eight-column table still scrolls without squeezing ordinary values onto multiple lines.
  • Lint and production build pass. Two local full-suite runs hit unchanged MCP startup/timing assertions (2281/2283 and 2282/2283); the unchanged MCP unit and integration files pass separately, 51/51 and 10/10. No existing assertions, timeouts, concurrency settings, or skips were changed. Official CI passes on Node 22.19.0: 2283/2283 tests (0 skipped), lint, build, and all 12 browser checks.

Merge Danger

Door: two-way
Blast Radius: Markdown

Bound header and data cells while preserving natural table sizing and horizontal scrolling. Cover long headings, prose, links, inline code, and wide tables in the existing desktop and mobile browser regression.

Fixes agegr#1052.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

This branch has not been deployed

No deployments
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.

Few-column tables with long-sentence cells never wrap since #650 (width: max-content) — clipped text + forced horizontal scrolling

1 participant