Repository navigation

board: review gate comments edit on click, one card per finding - #947

Merged

m4ttheweric merged 88 commits into

mainfrom

comment-click-to-edit

Oct 28, 2026

Merged

board: review gate comments edit on click, one card per finding#947
m4ttheweric merged 88 commits into
mainfrom
comment-click-to-edit

Commits on Oct 28, 2026
- board: map a click in rendered markdown to its source offset
  
  m4ttheweric
  and
  claude
  committed
  07361cc
- board: click-to-edit comment field, first pass in Storybook
  
  m4ttheweric
  and
  claude
  committed
  ce53416
- board: tint a summary field while it edits
  
  m4ttheweric
  and
  claude
  committed
  71c7126
- board: a summary field edits on the neutral wash, not green
  
  m4ttheweric
  and
  claude
  committed
  367f278
- board story: stripe the summary line, more room between summary fields
  
  m4ttheweric
  and
  claude
  committed
  33a3869
- board: summary fields hug their text; the space between them is margin
  
  m4ttheweric
  and
  claude
  committed
  12fb177
- board: ease the click-to-edit field's change over 150ms
  
  m4ttheweric
  and
  claude
  committed
  7ffccee
- board: add motion 14.2.0
  
  m4ttheweric
  and
  claude
  committed
  11aaafe
- board: crossfade read and write with motion, ease the height; roomier write field
  
  m4ttheweric
  and
  claude
  committed
  eea04f3
- board story: diff-line findings edit in the borderless field too
  
  m4ttheweric
  and
  claude
  committed
  d7e11ba
- board: a comment's last block drops its bottom margin
  
  m4ttheweric
  and
  claude
  committed
  d4dfae8
- board: even the rest-state field's padding around inline code
  
  m4ttheweric
  and
  claude
  committed
  f342592
- board: balance the rest-state wash on the ink, not the line box
  
  m4ttheweric
  and
  claude
  committed
  3a1347a
- board: even the rest-state wash around the letters themselves
  
  m4ttheweric
  and
  claude
  committed
  28126cb
- board: more room under a rest-state comment
  
  m4ttheweric
  and
  claude
  committed
  52fd6bf
- board: 20px under a rest-state comment
  
  m4ttheweric
  and
  claude
  committed
  a7b9640
- board: back to 8px under a rest-state comment
  
  m4ttheweric
  and
  claude
  committed
  b831a83
- board: 8px above and below a rest-state comment
  
  m4ttheweric
  and
  claude
  committed
  6547245
- board: review sheet comments read rendered and edit on a click
  
  m4ttheweric
  and
  claude
  committed
  29eefae
- board story: the drafted first round shows its diffs
  
  m4ttheweric
  and
  claude
  committed
  2fca846
- board: a finding's comment edits in a filled well, no stripe
  
  m4ttheweric
  and
  claude
  committed
  7de0a86
- board: the summary card holds the summary line only, in the filled field
  
  m4ttheweric
  and
  claude
  committed
  47f6e69
- board: the summary badge reads review verdict: <verdict>
  
  m4ttheweric
  and
  claude
  committed
  17138c2
- board: the re-review ask is a row in the respond dock
  
  m4ttheweric
  and
  claude
  committed
  ed97f35
- board: drop the review dock's global reset
  
  m4ttheweric
  and
  claude
  committed
  4e8d15b
- board: code fences name their language (TypeScript, not ts)
  
  m4ttheweric
  and
  claude
  committed
  1c6a86f
- design: review gate boards B23-B34 (summary, click to edit, laptop rail, one box per finding)
  
  m4ttheweric
  and
  claude
  committed
  5cf295a
- board: review cards are one box per finding; summary moves to the dock
  
  m4ttheweric
  and
  claude
  committed
  cc9ebcf
- board: a laptop-sized rail: checks fold to a tally, MR card to three lines
  
  m4ttheweric
  and
  claude
  committed
  d608cda
- board: review dock and context match the design
  
  m4ttheweric
  and
  claude
  committed
  0dbbffc
- board: the whole card head toggles its finding; drop the finding mark
  
  m4ttheweric
  and
  claude
  committed
  992acbf
- board: a finding with no diff line wears an in-summary badge
  
  m4ttheweric
  and
  claude
  committed
  308436e
- board: the card head hovers in the finding rows' blue wash
  
  m4ttheweric
  and
  claude
  committed
  887c110
- board tests: assert the redesigned review gate (click-to-edit, dock summary, flat drafted list)
  
  m4ttheweric
  and
  claude
  committed
  774b5f1
- board: a drafted comment is a plain comment box; Escape stays closed
  
  m4ttheweric
  and
  claude
  committed
  cb3a7bf
- board: a drafted comment is a field on a grey comment area (B38)
  
  m4ttheweric
  and
  claude
  committed
  fca34de
- board: soften the comment field and its hover hint
  
  m4ttheweric
  and
  claude
  committed
  ca505e8
- board: 'Click to edit' is a pill on the comment field's border
  
  m4ttheweric
  and
  claude
  committed
  de5bcac
- board: the click-to-edit pill wears the accent wash
  
  m4ttheweric
  and
  claude
  committed
  45185d0
- board: the click-to-edit pill gets a pencil, a centered wash and room
  
  m4ttheweric
  and
  claude
  committed
  ec097b9
- board: drop the comment field's hover glow; the pill carries hover
  
  m4ttheweric
  and
  claude
  committed
  da9fe66
- board: keyboard focus on a comment rings its field, not a clipped outline
  
  m4ttheweric
  and
  claude
  committed
  6648da6
- board: closing a changed comment flashes 'Comment updated' in its pill
  
  m4ttheweric
  and
  claude
  committed
  ed61028
- board: a click that closes a comment doesn't also open the full diff
  
  m4ttheweric
  and
  claude
  committed
  2b297e1
- board: closing a comment no longer flashes the click-to-edit pill
  
  m4ttheweric
  and
  claude
  committed
  05a2782
- board: 'Comment updated' wears the ok wash
  
  m4ttheweric
  and
  claude
  committed
  7996414
- board: the comment pill transitions all over 150ms
  
  m4ttheweric
  and
  claude
  committed
  2f211bc
- board: the updated flash holds its label for the pill's 150ms fade
  
  m4ttheweric
  and
  claude
  committed
  1964216
- board: the comment pill eases in and out with a small lift
  
  m4ttheweric
  and
  claude
  committed
  fd2ab3e
- board: full diff reveals on hovering the code section, not the whole card
  
  m4ttheweric
  and
  claude
  committed
  c645a42
- board: an edited draft shows one 'Reset to draft' button, no dotted pair
  
  m4ttheweric
  and
  claude
  committed
  a47f947
- board: tuck the write hint closer under the comment field
  
  m4ttheweric
  and
  claude
  committed
  56a7d44
- board: a comment card eases its height on reset and edit
  
  m4ttheweric
  and
  claude
  committed
  a22150a
- board: a comment's header says it's being edited and holds the reset
  
  m4ttheweric
  and
  claude
  committed
  acb57db
- board: the comment header shows only while writing
  
  m4ttheweric
  and
  claude
  committed
  cdcce69
- board: an edited comment wears an 'Edited' pill with its reset at rest
  
  m4ttheweric
  and
  claude
  committed
  2cef85c
- board: polish the edited pill: split state and reset, text clears it
  
  m4ttheweric
  and
  claude
  committed
  a7d2e91
- board: the edited pill's reset hovers by deepening, no underline or wash
  
  m4ttheweric
  and
  claude
  committed
  e83ad8b
- board: the edited pill's reset hovers with a full-half blue wash
  
  m4ttheweric
  and
  claude
  committed
  7c0c330
- board: the edited pill's hairline fades while its reset is washed
  
  m4ttheweric
  and
  claude
  committed
  8754194
- board: comment code blocks are a frameless grey wash with a floated label
  
  m4ttheweric
  and
  claude
  committed
  90ed1db
- board: highlighted comment code sits in the wash with its label clear
  
  m4ttheweric
  and
  claude
  committed
  735fa9e
- board: comment code wash takes 4px corners
  
  m4ttheweric
  and
  claude
  committed
  9f1aef6
- board: comment code drops the highlighter's scrollbar gutter
  
  m4ttheweric
  and
  claude
  committed
  f5c938e
- board: review findings cap at 960px, centred in the main column
  
  m4ttheweric
  and
  claude
  committed
  c93ffc9
- board: wrapped inline code closes each piece; findings cap at 1120px
  
  m4ttheweric
  and
  claude
  committed
  a67c824
- board: review dock back to its earlier verdict choices, summary kept
  
  m4ttheweric
  and
  claude
  committed
  fdd9fbf
- board: the dock's summary reads 'Review comment'
  
  m4ttheweric
  and
  claude
  committed
  cf6bacf
- board: unpicked dock choices are white cards on the grey dock
  
  m4ttheweric
  and
  claude
  committed
  5d3ff7e
- board story: drafted first round serves a report, so the record shows
  
  m4ttheweric
  and
  claude
  committed
  df46f6e
- board: the review record sits in a card under a 'Review report' heading
  
  m4ttheweric
  and
  claude
  committed
  a4096dc
- board: the review main column splits into Findings and Report tabs
  
  m4ttheweric
  and
  claude
  committed
  9a34582
- board: checks move from the rail into the Report tab's card
  
  m4ttheweric
  and
  claude
  committed
  1995033
- board: checks stay in the rail too, beside the Report tab's row
  
  m4ttheweric
  and
  claude
  committed
  8feed79
- board: the rail's checks card lists every check, no fold
  
  m4ttheweric
  and
  claude
  committed
  5b642bd
- board: a finding with no diff line shows the agent's finding in the code's slot
  
  m4ttheweric
  and
  claude
  committed
  48ca0a0
- board: tidy the no-diff finding: one border, full-strength 14px text
  
  m4ttheweric
  and
  claude
  committed
  00ff4bc
- board: the no-diff finding sits on a light blue wash
  
  m4ttheweric
  and
  claude
  committed
  5474bee
- board: an info mark in the code's file bar opens the agent's finding on hover
  
  m4ttheweric
  and
  claude
  committed
  f3d84d1
- board: the finding info mark opens on click, with a pointer cursor
  
  m4ttheweric
  and
  claude
  committed
  723ea64
- board: folding a finding's code eases its height over 150ms
  
  m4ttheweric
  and
  claude
  committed
  db0e841
- board: drop the findings tally's 'N more below' and its scroll tracking
  
  m4ttheweric
  and
  claude
  committed
  c89d01a
- board: the drafted findings tally reads 'Posting N of M'
  
  m4ttheweric
  and
  claude
  committed
  9a88e2a
- board: ticking a finding eases its card open and closed
  
  m4ttheweric
  and
  claude
  committed
  1756acc
- board tests: follow the review gate's final UI
  
  m4ttheweric
  and
  claude
  committed
  b1f8d77
- board: fence labels read own keys only, so __proto__ can't crash a comment
  
  m4ttheweric
  and
  claude
  committed
  40f8da7
- board: a tall dock scrolls inside the rail, submit stuck to its bottom
  
  m4ttheweric
  and
  claude
  committed
  a3b84c2
- board tests: review-code-dom skips motion's animations under happy-dom
  
  m4ttheweric
  and
  claude
  committed
  cfca404

Commits

Commits on Oct 28, 2026