Skip to content

fix(web): keep a document link inline in the coordinator's sentence - #183

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-12088-keep-a-document-link-inline-in-the-coordinators-sentence
Nov 4, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-12088-keep-a-document-link-inline-in-the-coordinators-sentence

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

A document name in a conversation reply rendered as a button styled with inline-flex, a 44px minimum height and side padding. A button is laid out as one unbreakable box, so the title dropped onto its own line, the line grew taller and the punctuation after it stood apart. The name now renders as an inline link that wraps with the sentence and still opens the document on click. The print view no longer needs to spare the mention when it strips buttons.

Verified with a new transcript test that checks the bold title is an inline link with the period attached and that clicking it opens the document, plus before and after screenshots of the real conversation component at full and narrow widths in headless Chromium. Lint, format check, typecheck, the test suite, pins:verify and vendor:verify pass.

Closes CL-12088

@linear-code

linear-code Bot commented Nov 4, 2026

Copy link
Copy Markdown

CL-12088

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Real RoomThread with app CSS at 720px and 360px (gist).

Before (main) After (this PR)
before after

@TheGreatAxios
TheGreatAxios marked this pull request as ready for review November 4, 2026 23:25
@corbits-triage

corbits-triage Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Merged before Triage finished on f53ceb7.

Comment thread apps/web/src/markdown.tsx Resolved
corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 4, 2026
@TheGreatAxios
TheGreatAxios force-pushed the cl-12088-keep-a-document-link-inline-in-the-coordinators-sentence branch from f725bb9 to f53ceb7 Compare November 4, 2026 23:45
@TheGreatAxios
TheGreatAxios merged commit 66285ae into main Nov 4, 2026
9 checks passed
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.

1 participant