Skip to content

Commit c3e940d

Browse files
committed
style(plan-diff): refine modified-block visual — amber gutter, no fill
Drop the yellow background fill from .plan-diff-modified and keep only a softened amber left border. Added/removed blocks remain loud (full fill + strong border) because add/remove are block-scope events — the whole block matters. Modify is a word-scope event — the individual changed words carry loud inline red/green highlights, and a block-level fill would compete with that inline work. The amber gutter at 75% opacity now reads as a quiet "look inside, the change is in the text" marker that sits coherently with the rest of the palette. For provenance purposes, this commit was AI assisted.
1 parent 9dd0031 commit c3e940d

1 file changed

Lines changed: 10 additions & 7 deletions

File tree

packages/editor/index.css

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -149,18 +149,21 @@ pre code.hljs .hljs-code {
149149
}
150150

151151
/* Clean diff view - modified content (mix of additions and deletions in one
152-
block, rendered inline via word-level diff). Amber/yellow border matches
153-
the GitHub / VSCode convention that green=add, red=remove, yellow=both. */
152+
block, rendered inline via word-level diff).
153+
Deliberate asymmetry with added/removed: add/remove are BLOCK-scope events
154+
— the whole block matters, so a loud fill is the right signal. Modify is
155+
a WORD-scope event — the words matter, and the inline red-struck /
156+
green-highlighted word markers already grab attention. A block-level fill
157+
would compete with that inline work; an amber gutter on a normal
158+
background says "look inside, the change is in the text" while staying
159+
consistent with the green/red/yellow diff convention. */
154160
.plan-diff-modified {
155-
border-left: 3px solid var(--warning);
156-
background: oklch(from var(--warning) l c h / 0.06);
161+
border-left: 3px solid oklch(from var(--warning) l c h / 0.75);
162+
background: transparent;
157163
padding-left: 0.75rem;
158164
border-radius: 0 0.25rem 0.25rem 0;
159165
margin: 0.25rem 0;
160166
}
161-
.light .plan-diff-modified {
162-
background: oklch(from var(--warning) l c h / 0.06);
163-
}
164167

165168
/* Clean diff view - unchanged (dimmed) */
166169
.plan-diff-unchanged {

0 commit comments

Comments
 (0)