Repository navigation
Conversation
External link text gets a <wbr> after every character so long URLs can wrap. In an expanded table on a wide lane the columns may shrink to their min-content width, and those break opportunities let a short link such as pingdotgg#363 collapse to "pingdotgg#36" and "3" on two lines. Hide the <wbr> inside the wide-lane table rules so the longest word sets the column width, as the table's own wrapping rules intend. Long link text still wraps through the cell's overflow-wrap. Fixes pingdotgg#24146
github-actions
Bot
added
vouch:unvouched
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — Adds a narrowly scoped CSS rule that prevents character-level wrapping of external links in expanded Markdown tables while preserving existing behavior elsewhere. The change is isolated to presentation and does not affect data processing, configuration, or product defaults. You can add or adjust custom eligibility rules. Learn more. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly and concisely describes the CSS fix that prevents PR links in expanded tables from breaking mid-number. |
| Description check | |
The description covers the problem, cause, change, scope, linked issue, verification results, screenshots, and known limitations. It provides sufficient detail for review. |
| Linked Issues check | |
Issue #24146 is the active direct linked issue. The supplied PR evidence shows a rule in apps/web/src/index.css within the wide expanded-table container that hides wbr elements in th and td ce… |
| Out of Scope Changes check | |
The supplied change summary contains only six added CSS lines in apps/web/src/index.css. The rule targets expanded Markdown table cells in the existing chat-lane (min-width: 36rem) scope. The chan… |
Full details: Linked Issues check
-
Explanation
Issue
#24146is the active direct linked issue. The supplied PR evidence shows a rule inapps/web/src/index.csswithin the wide expanded-table container that hideswbrelements inthandtdcells. This removes character-level break opportunities for PR link text. The reported 736px verification changed#363from two lines to one line. The existingoverflow-wrap: break-wordremains available for long cell text. The reported format and typecheck checks passed. No issue requirement for an automated layout test is stated.
Full details: Out of Scope Changes check
-
Explanation
The supplied change summary contains only six added CSS lines in
apps/web/src/index.css. The rule targets expanded Markdown table cells in the existingchat-lane (min-width: 36rem)scope. The change directly addresses issue#24146and does not add unrelated behavior, settings, or files.
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
Problem
In an expanded Markdown table on a wide chat lane, a PR link chip such as
#363can break in the middle of its number, so#36and3end up on two lines. Repro: render a table whose first column holds short PR links and whose other columns hold long prose, then expand it on a lane at least 36rem wide.Change
MarkdownExternalLinkputs a<wbr>after every character of the link text so that long URLs can wrap. Those break opportunities bring the link's min-content width down to a single character. Under@container chat-lane (min-width: 36rem)(#20878), the expanded table usesmin-width: 0, so the first column can shrink to that width and the chip splits.overflow-wrap: break-wordontdis not the cause, because it doesn't change min-content.The fix is one rule inside that same container block: hide
wbrin expanded table cells. The longest word then sets the column width, which is what the table's wrapping rules intend. Long link text in a cell still wraps through the existingoverflow-wrap: break-word. Links outside tables are unchanged.apps/web/src/index.css: +6/−0.Scope and approval
Closes #24146
This is a small, focused CSS fix for a visible layout bug. It touches only the existing wide-lane expanded-table rules and adds no new behavior or settings.
Verification
Tested in the web dev app with system Chrome (headless, 1600×1000 at 2x), using a fake empty project and one seeded assistant message holding the table. No provider was called. Lane width 736px, table
data-expanded="true".main)#36/3#363I also checked a static page with a very long URL and a long unbreakable word in column 1. On a 720px lane the table still did not overflow horizontally, and the column capped at the cell's
max-width: 24rem.Before (light):
After (light):
Before (dark):
After (dark):
Checks:
vp fmt --check apps/web/src/index.css: passes.vp run --filter @t3tools/web typecheck: exit 0.vp run check:changed: passes (format, typecheck).There's no unit test because this is a CSS-only layout change, and node tests don't cover layout.
Not checked:
thcell containing a link, since onlytdwas exercised.Claude Sonnet 5.5 (xhigh) via T3 Code.