prreviewbuddy 0.52.2 → 0.53.1

package/static/shell.css

@@ -3177,8 +3177,7 @@ a.prb-rail-item { text-decoration: none; }
 .prb-index[data-selecting] .prb-card-link { padding-left: 42px; }
 /* A ticked row is legible without finding its box, which matters most on the rows a search has
    just brought back into view. */
-.prb-card[data-selected] { border-color: var(--brand); }
-.prb-card[data-selected] .prb-card-link { background: var(--brand-muted); }
+.prb-card[data-selected] { border-color: var(--brand); background: var(--brand-muted); }
 /* Nothing in the row navigates while selecting, so nothing in it should look like it might. */
 .prb-index[data-selecting] .prb-card-link { cursor: default; }
 
@@ -4543,7 +4542,7 @@ a.prb-rail-item { text-decoration: none; }
   line-height: var(--lh-body);
   color: var(--pane-ink-muted);
 }
-.prb-kiss-kept-title { margin: 0 0 4px; font-weight: 600; color: var(--pane-ink); }
+.prb-kiss-kept-title { font-weight: 600; color: var(--pane-ink); }
 
 /* What validation withdrew, in the amber the disclosure strip already uses for exactly this. */
 .prb-kiss-audit {
@@ -4668,29 +4667,153 @@ a.prb-rail-item { text-decoration: none; }
 .prb-kiss-evidence .prb-kiss-locations { margin-top: 4px; }
 .prb-kiss-draft blockquote { margin: 0; font-size: var(--t-body); line-height: var(--lh-body); color: var(--pane-ink); }
 
-/* The same control the assistant's replies carry, in this pane's light palette. Literal colours
-   rather than the shell's variables, because the document is light on every theme and a token
-   defined against the dark shell would put grey on white here. */
-.prb-kiss-draft .pr-helper-chat-actions { display: flex; margin-top: 8px; }
-.prb-kiss-draft .pr-helper-msg-copy {
-  display: inline-flex;
+/* ── KISS decisions ──
+   The card's title row holds Copy and Ask Buddy as a finding's does. The footer holds what acts on
+   the recommendation: Open file and Copy draft comment on the left, the decision on the right. */
+.prb-kiss-rec-title-row { display: flex; align-items: flex-start; gap: 8px; }
+.prb-kiss-rec-title-row > .prb-kiss-rec-title { flex: 1; min-width: 0; }
+.prb-kiss-rec-title:focus { outline: none; }
+.prb-kiss-rec-title:focus-visible { outline: 2px solid var(--pane-accent); outline-offset: 2px; border-radius: 4px; }
+
+.prb-kiss-decision-status {
+  padding: 2px 8px;
+  border-radius: 999px;
+  font-size: var(--t-micro);
+  white-space: nowrap;
+  background: rgba(15, 23, 42, 0.05);
+  color: var(--pane-ink-muted);
+}
+/* Accepted in the pane's accent, declined in neutral: two ordinary answers, neither of them a
+   warning. Green would say "fixed", which a decision about a suggestion is not. */
+.prb-kiss-decision-status[data-kiss-status='accepted'] { background: rgba(147, 51, 234, 0.1); color: var(--pane-accent-ink); }
+.prb-kiss-decision-status[data-kiss-status='declined'] { background: rgba(15, 23, 42, 0.08); color: var(--pane-ink); }
+.prb-kiss-stale {
+  padding: 1px 7px;
+  border-radius: 999px;
+  border: 1px solid rgba(180, 83, 9, 0.3);
+  font-size: var(--t-micro);
+  color: #9a3412;
+  white-space: nowrap;
+}
+
+.prb-kiss-rec-actions {
+  display: flex;
+  flex-wrap: wrap;
   align-items: center;
-  gap: 5px;
-  padding: 3px 6px;
-  border: 0;
-  border-radius: 5px;
-  background: none;
+  justify-content: space-between;
+  gap: 8px 12px;
+  margin-top: 14px;
+  padding-top: 12px;
+  border-top: 1px solid var(--pane-line);
+}
+.prb-kiss-rec-actions-aside { display: flex; flex-wrap: wrap; align-items: center; gap: 4px 10px; }
+.prb-kiss-decide-group { display: inline-flex; flex-wrap: wrap; gap: 6px; margin-left: auto; }
+
+.prb-kiss-open-file,
+.prb-kiss-decide,
+.prb-kiss-run-again,
+.prb-kiss-fresh-run,
+.prb-kiss-run-cancel,
+.prb-kiss-decision-retry {
+  padding: 5px 12px;
+  border-radius: 6px;
+  border: 1px solid var(--pane-line);
+  background: var(--pane-surface);
+  font: inherit;
+  font-size: var(--t-meta);
+  font-weight: 600;
   color: var(--pane-ink-muted);
-  font-family: inherit;
-  font-size: 11px; /* icon button, sized to its glyph, not the type scale */
+  white-space: nowrap;
+  cursor: pointer;
+}
+.prb-kiss-open-file:hover,
+.prb-kiss-decide:hover:not(:disabled),
+.prb-kiss-run-again:hover,
+.prb-kiss-fresh-run:hover,
+.prb-kiss-run-cancel:hover,
+.prb-kiss-decision-retry:hover { border-color: rgba(15, 23, 42, 0.3); color: var(--pane-ink); }
+.prb-kiss-decide[aria-pressed='true'] { border-color: rgba(147, 51, 234, 0.45); background: rgba(147, 51, 234, 0.08); color: var(--pane-accent-ink); }
+.prb-kiss-decide:disabled { cursor: progress; opacity: 0.6; }
+:is(.prb-kiss-open-file, .prb-kiss-decide, .prb-kiss-run-again, .prb-kiss-fresh-run, .prb-kiss-run-cancel, .prb-kiss-decision-retry):focus-visible {
+  outline: 2px solid var(--pane-accent);
+  outline-offset: 1px;
+}
+
+/* Copy draft comment and Copy recommendations: the finding copy control with its word showing.
+   The word is the label, so the hover bubble that repeats it is not drawn; after a click the
+   word itself says Copied or Copy failed. */
+.prb-doc :is(.prb-kiss-rec-actions, .prb-kiss-handoff) .pr-helper-msg-copy.pr-helper-finding-copy {
+  width: auto;
+  height: auto;
+  margin: 0;
+  gap: 5px;
+  padding: 5px 8px;
+  font-size: var(--t-meta);
   font-weight: 600;
-  line-height: 1;
+  color: var(--pane-ink-muted);
+}
+.prb-doc :is(.prb-kiss-rec-actions, .prb-kiss-handoff) .pr-helper-finding-copy::after { display: none; }
+.prb-doc :is(.prb-kiss-rec-actions, .prb-kiss-handoff) .pr-helper-msg-copy .pr-helper-msg-copy-label[hidden] { display: none; }
+.prb-doc :is(.prb-kiss-rec-actions, .prb-kiss-handoff) .pr-helper-msg-copy.copied { color: var(--pane-success); }
+.prb-doc :is(.prb-kiss-rec-actions, .prb-kiss-handoff) .pr-helper-msg-copy.failed { color: var(--pane-danger); }
+.prb-doc .prb-kiss-handoff .pr-helper-msg-copy.pr-helper-finding-copy { border: 1px solid var(--pane-line); background: var(--pane-surface); }
+.prb-doc .prb-kiss-handoff .pr-helper-msg-copy:disabled { cursor: default; opacity: 0.5; }
+
+.prb-kiss-decision-error { flex-basis: 100%; margin: 0; font-size: var(--t-meta); color: #9a3412; }
+.prb-kiss-decision-retry { margin-left: 6px; padding: 2px 8px; }
+
+.prb-kiss-handoff {
+  display: flex;
+  flex-wrap: wrap;
+  align-items: center;
+  gap: 6px 12px;
+  margin: 0 0 16px;
+}
+.prb-kiss-handoff-note { margin: 0; font-size: var(--t-meta); line-height: var(--lh-body); color: var(--pane-ink-muted); }
+.prb-kiss-progress { margin: 0 0 14px; }
+.prb-kiss-decided { margin-top: 8px; }
+
+/* Whose code the run was about, as a fact of the run beside its other facts. */
+.prb-kiss-perspective { margin-left: auto; font-size: var(--t-meta); color: var(--pane-ink-muted); white-space: nowrap; }
+.prb-doc .prb-doc-head .prb-kiss-perspective + .prb-doc-provenance { margin-left: 12px; }
+.prb-doc .prb-doc-head .prb-kiss-run-again { align-self: center; margin-left: 8px; padding: 3px 10px; }
+.prb-doc .prb-doc-head .prb-kiss-run-again + .prb-focus-toggle { margin-left: 0; }
+
+.prb-kiss-run-dialog {
+  width: min(440px, calc(100vw - 32px));
+  padding: 18px 20px;
+  border: 1px solid var(--pane-line);
+  border-radius: 12px;
+  background: var(--pane-surface);
+  color: var(--pane-ink);
+}
+.prb-kiss-run-dialog::backdrop { background: rgba(15, 23, 42, 0.35); }
+.prb-kiss-run-dialog-title { margin: 0 0 10px; font-size: var(--t-section); font-weight: 600; }
+.prb-kiss-run-dialog .prb-kiss-run { margin: 0; }
+.prb-kiss-run-cancel { margin-top: 10px; }
+
+.prb-kiss-fresh-actions { display: flex; flex-wrap: wrap; gap: 8px; margin-top: 10px !important; }
+.prb-kiss-fresh-actions .prb-kiss-fresh-dismiss { margin-top: 0; }
+.prb-kiss-fresh[data-collapsed] { padding: 8px 12px; }
+.prb-kiss-fresh[data-collapsed] p { font-size: var(--t-meta); }
+.prb-kiss-fresh[data-collapsed] .prb-kiss-fresh-run { margin-left: 6px; padding: 2px 8px; }
+
+/* Complexity worth keeping: folded twice. The heading says considered reasoning is here; opened,
+   each item is its decision and its reason on one row; one more click is the rationale and files. */
+.prb-kiss-justified { margin: 22px 0 0; }
+.prb-kiss-justified-summary {
+  font-size: var(--t-pill);
+  font-weight: 700;
+  letter-spacing: 0.07em;
+  text-transform: uppercase;
+  color: var(--pane-ink-muted);
   cursor: pointer;
 }
-.prb-kiss-draft .pr-helper-msg-copy:hover { color: var(--pane-ink); background: rgba(31, 35, 40, 0.06); }
-.prb-kiss-draft .pr-helper-msg-copy.copied { color: var(--pane-success); }
-.prb-kiss-draft .pr-helper-msg-copy.failed { color: var(--pane-danger); }
-.prb-kiss-draft .pr-helper-msg-copy svg { display: block; }
+.prb-kiss-justified[open] > .prb-kiss-justified-summary { margin-bottom: 10px; }
+.prb-kiss-kept > .prb-kiss-kept-summary { cursor: pointer; list-style-position: outside; }
+.prb-kiss-kept-summary .prb-kiss-kept-title { margin: 0; }
+.prb-kiss-kept-reason { display: block; margin-top: 2px; color: var(--pane-ink-muted); font-weight: 400; }
+.prb-kiss-kept[open] > .prb-kiss-kept-summary { margin-bottom: 8px; }
 
 .prb-kiss-locations { margin: 12px 0 0; }
 .prb-kiss-locations-list { margin: 0; padding: 0; list-style: none; }
@@ -5185,7 +5308,7 @@ body .prb-body[data-focus='buddy'] .prb-msg-user { max-width: min(90%, 78ch); }
    Copy. Positioned as the flex child it is, the way `.prb-header .prb-menu` is, so the panel hangs
    off the trigger rather than off a corner of the card. The panel is in the card's flow inside the
    scrolling pane, so near the bottom it extends the scroll rather than being cut off. */
-.prb-doc .pr-helper-issue-title-row .prb-menu {
+.prb-doc :is(.pr-helper-issue-title-row, .prb-kiss-rec-title-row) .prb-menu {
   position: relative;
   top: auto;
   right: auto;