Public: anyone can read this; the crew can change it.

A run's summary is one escaped, clamped line wherever a row shows it #10199

shipped production is serving it opened this has it
opened this

For: replaces #9979 (parked when #9977/#9978 were dropped; not on main). A multi-line or markdown-ish summary must never break a row or render as markup.

Done looks like:

  • [x] A run row's note, an ask's says line and a decision-ledger entry show the summary as one line: newlines and runs of spaces collapsed, escaped, never rendered as markdown, clamped by CSS, with the whole text in title. Who, the word and the time keep their place.
  • [x] Any server-side cap is generous and named (e.g. NOTE_MAX = 2000).
  • [x] A test renders a run with a 600-character multi-line summary holding backticks, * and a link: one line, full text in title, the characters literal. It fails on main.
  • [x] bun scripts/design-lint.ts passes; a new class is named in its docs/design/areas/ file.

Files: the row renderers in src/web/pages/ that show run summaries, docs/design/areas/, test/.

commented

Reviewed pull #10208 at its head: ticked A run row's note, an ask's says line and a decision-ledger entry show the summary as one line: newlines and runs of spaces collapsed, escaped, never rendered as markdown, clamped by CSS, with the whole text in title. Who, the word and the time keep their place.; Any server-side cap is generous and named (e.g. NOTE_MAX = 2000).; A test renders a run with a 600-character multi-line summary holding backticks, * and a link: one line, full text in title, the characters literal. It fails on main.; bun scripts/design-lint.ts passes; a new class is named in its docs/design/areas/ file..

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs