prreviewbuddy 0.85.0 → 0.86.6

package/static/shell.css

@@ -44,6 +44,8 @@
   --brand:          #9333ea;
   --brand-hover:    light-dark(#7427e8, #7c3aed);
   --brand-muted:    light-dark(#f3e8ff, rgba(147, 51, 234, 0.15));
+  /* A brand outline that marks a surface as the selection's without shouting: the bulk bar. */
+  --brand-border:   light-dark(rgba(147, 51, 234, 0.35), rgba(167, 112, 245, 0.45));
   /* Brand-coloured words on a light or tinted ground, where #9333ea itself is too pale to read. */
   --brand-ink:      light-dark(#6b21a8, #c4a5f5);
   /* Text and marks on a filled --brand control. White in both, because the fill is the same. */
@@ -1831,6 +1833,36 @@ a.prb-rail-item { text-decoration: none; }
 .pr-helper-explain-simply-button:disabled { opacity: 0.55; cursor: default; }
 .pr-helper-explain-simply-button:disabled:hover { background: transparent; }
 
+/* What a finished turn offers under its answer: Reconcile with findings, or a reconcile's Accept all.
+   Quiet, the size of the answer's own meta, so a long conversation does not read as a column of
+   buttons; the one sentence beside it says what the action is for. */
+.prb-turn-actions {
+  display: flex;
+  flex-wrap: wrap;
+  align-items: baseline;
+  gap: 4px 10px;
+  margin: -2px 0 10px;
+}
+.prb-turn-reconcile,
+.prb-turn-accept-all {
+  padding: 3px 10px;
+  border: 1px solid var(--border);
+  border-radius: 8px;
+  background: transparent;
+  color: var(--text-secondary);
+  font: inherit;
+  font-size: var(--t-micro);
+  cursor: pointer;
+}
+.prb-turn-accept-all { border-color: var(--brand-hover); color: var(--text-primary); font-weight: 600; }
+.prb-turn-reconcile:hover:not(:disabled),
+.prb-turn-accept-all:hover:not(:disabled) { border-color: var(--border-strong); color: var(--text-primary); }
+.prb-turn-reconcile:focus-visible,
+.prb-turn-accept-all:focus-visible { outline: 2px solid var(--brand-hover); outline-offset: 2px; }
+.prb-turn-reconcile:disabled,
+.prb-turn-accept-all:disabled { opacity: 0.55; cursor: default; }
+.prb-turn-help { margin: 0; color: var(--text-secondary); font-size: var(--t-micro); }
+
 /* ── The reviewer's own prompts ──
    A strip between the assistant's header and its conversation: reachable without scrolling the
    log, and out of the composer, which is already carrying the scope, the field and the button.
@@ -1903,6 +1935,8 @@ a.prb-rail-item { text-decoration: none; }
 
 /* The prompt editor is wider than the shutdown dialog: it holds a paragraph, not a sentence. */
 .prb-prompt-dialog { max-width: 560px; width: calc(100vw - 48px); }
+.prb-prompt-error { margin: 0; font-size: var(--t-meta); color: var(--danger-ink); }
+.prb-prompt-error[hidden] { display: none; }
 .prb-field { display: block; margin-bottom: 12px; }
 .prb-field span {
   display: block;
@@ -3421,69 +3455,125 @@ a.prb-guide-pick { text-decoration: none; }
 
 /* ── The bulk bar ──
    Sticky at the foot of the column rather than under the list, because a selection made on page one
-   is acted on from wherever the reader has scrolled to. On the card surface with a brand border and
-   a shadow, so it reads as sitting above the list rather than as the last row of it. */
+   is acted on from wherever the reader has scrolled to. On the card surface with a faint brand
+   border and a float shadow, so it reads as sitting above the list rather than as the last row of
+   it. Two rows: what is selected, then what to do with it, the second only once something is. */
 .prb-bulk {
   position: sticky;
   bottom: 16px;
   z-index: 3;
   display: flex;
-  flex-wrap: wrap;
-  align-items: center;
-  gap: 12px;
-  margin-top: 16px;
-  padding: 12px 16px;
-  border: 1px solid var(--brand);
+  flex-direction: column;
+  gap: 10px;
+  max-width: 880px;
+  margin: 16px auto 0;
+  padding: 10px 10px 10px 16px;
+  border: 1px solid var(--brand-border);
   border-radius: 12px;
   background: var(--surface-raised);
-  box-shadow: 0 6px 20px var(--shadow-soft);
+  box-shadow: 0 12px 32px var(--shadow-float), 0 2px 6px var(--shadow-soft);
 }
 .prb-bulk[hidden] { display: none; }
-.prb-bulk-count {
-  margin: 0;
-  font-size: var(--t-meta);
-  color: var(--text-secondary);
-}
+.prb-bulk-head { display: flex; align-items: center; gap: 8px; min-height: 32px; }
+.prb-bulk-status { flex: 1; min-width: 0; margin: 0; font-size: var(--t-meta); color: var(--text-muted); }
+.prb-bulk-count { font-size: var(--t-body); font-weight: 600; color: var(--text-primary); }
+.prb-bulk-detail { margin-left: 4px; }
+.prb-bulk-detail:empty { display: none; }
 /* The `display: flex` below outranks the sheet's `[hidden]` rule, so hiding this row for the
    confirmation needs saying here or the buttons stay under the question they replaced. */
-.prb-bulk-actions[hidden] { display: none; }
+.prb-bulk-actions[hidden],
+.prb-bulk[data-empty] .prb-bulk-actions { display: none; }
+.prb-bulk-actions,
+.prb-bulk-confirm {
+  padding: 10px 0 0;
+  border-top: 1px solid var(--border);
+}
 .prb-bulk-actions {
   display: flex;
   flex-wrap: wrap;
   align-items: center;
-  gap: 6px;
-  margin-left: auto;
+  gap: 8px;
 }
 .prb-bulk-button {
-  padding: 6px 12px;
-  border: 1px solid var(--border);
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+  box-sizing: border-box;
+  height: 32px;
+  padding: 0 12px;
+  border: 1px solid var(--border-strong);
   border-radius: 8px;
   background: transparent;
   color: var(--text-secondary);
   font: inherit;
   font-size: var(--t-meta);
+  white-space: nowrap;
   cursor: pointer;
 }
-.prb-bulk-button:hover:not(:disabled) { color: var(--text-primary); border-color: var(--border-strong); }
+.prb-bulk-button:hover:not(:disabled) { color: var(--text-primary); background: var(--wash); }
 .prb-bulk-button:disabled { opacity: 0.5; cursor: default; }
-/* Separated from the marks beside it, since this bar is mostly a workflow control and occasionally
-   a destructive one. */
-.prb-bulk-button-danger { margin-left: 6px; color: var(--danger); }
-.prb-bulk-button-danger:hover:not(:disabled) { border-color: var(--danger); color: var(--danger); }
-.prb-bulk-cancel { margin-left: 6px; }
+.prb-bulk-button[hidden] { display: none; }
+.prb-bulk-icon { display: inline-flex; flex: none; }
+.prb-bulk-icon svg { display: block; width: 16px; height: 16px; }
+/* The one filled control: what most selections are made for. */
+.prb-bulk-primary { border-color: var(--brand); background: var(--brand); color: var(--on-brand); font-weight: 600; }
+.prb-bulk-primary:hover:not(:disabled) { border-color: var(--brand-hover); background: var(--brand-hover); color: var(--on-brand); }
+/* The shared menu, placed as a control in the row rather than in a card's corner. Two classes,
+   because the base `.prb-menu` rule is further down the sheet and would win a tie. */
+.prb-bulk .prb-bulk-more { position: relative; top: auto; right: auto; z-index: auto; }
+.prb-bulk-more .prb-menu-panel { top: 38px; right: auto; left: 0; width: 220px; }
+.prb-bulk-more .prb-menu-panel[data-menu-placement='above'] { top: auto; bottom: 38px; }
+/* Its trigger's end edge, when hanging from its start would run off the right (menu.ts). */
+.prb-bulk-more .prb-menu-panel[data-menu-align='end'] { right: 0; left: auto; }
+/* Last and apart, quiet until it is reached for: a word in the danger ink, no outline. */
+.prb-bulk-button-danger { border-color: transparent; color: var(--danger-ink); }
+.prb-bulk-button-danger:hover:not(:disabled) { background: var(--danger-subtle); color: var(--danger-ink-strong); }
+.prb-bulk-actions > [data-bulk-delete] { margin-left: auto; }
+/* The way out, with its key beside it. The `<kbd>` is the command layer's (`showHint`), appended
+   after the icon and drawn before it. */
+.prb-bulk-exit {
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+  height: 32px;
+  padding: 0 6px;
+  border: 0;
+  border-radius: 8px;
+  background: transparent;
+  color: var(--text-muted);
+  cursor: pointer;
+}
+.prb-bulk-exit:hover:not(:disabled) { background: var(--surface-2); color: var(--text-primary); }
+.prb-bulk-exit:disabled { opacity: 0.5; cursor: default; }
+.prb-bulk-exit .prb-bulk-icon svg { width: 18px; height: 18px; }
+.prb-bulk-exit .prb-key-hint {
+  order: -1;
+  margin: 0;
+  padding: 1px 6px;
+  border: 1px solid var(--border);
+  border-radius: 5px;
+  background: var(--surface-2);
+  font: 500 var(--t-micro) / 1.5 ui-monospace, SFMono-Regular, 'SF Mono', Menlo, monospace;
+  opacity: 1;
+}
 /* A second state of the same bar, the way a card menu asks. */
-.prb-bulk-confirm { flex: 1 0 100%; }
 .prb-bulk-confirm[hidden] { display: none; }
 .prb-bulk-question { margin: 0 0 4px; font-size: var(--t-body); color: var(--text-primary); }
 .prb-bulk-note { margin: 0 0 10px; font-size: var(--t-meta); color: var(--text-tertiary, var(--text-secondary)); }
-.prb-bulk-confirm .prb-bulk-actions { margin-left: 0; }
+.prb-bulk-confirm-actions { display: flex; flex-wrap: wrap; gap: 8px; }
+.prb-bulk-confirm .prb-bulk-button-danger { border-color: var(--danger-border); background: var(--danger-subtle); }
 .prb-bulk-error {
-  flex: 1 0 100%;
   margin: 0;
   font-size: var(--t-meta);
   color: var(--danger);
 }
 .prb-bulk-error[hidden] { display: none; }
+/* A narrow column: the actions wrap, and Delete starts its own line's left edge rather than
+   stranding itself at the right of a short one. */
+@media (max-width: 640px) {
+  .prb-bulk { bottom: 8px; padding-left: 12px; }
+  .prb-bulk-actions > [data-bulk-delete] { margin-left: 0; }
+}
 /* ── A branch's history, inside its card ──
    Below the card's link and separated by a rule, because it is a different kind of thing: the link
    above is this branch as it stands, and this is how it got there. Folded away by default and
@@ -6552,6 +6642,18 @@ select.prb-addreview-control { padding-right: 32px; }
 .prb-addreview code,
 .prb-addreview pre { font-family: ui-monospace, SFMono-Regular, 'SF Mono', Menlo, monospace; }
 
+/* A finding's readable reference (I3, Q2): the name a feedback draft and the author's reply use for
+   it. Quiet beside the title, monospaced so I1 and I12 read as labels rather than words, and level
+   with the title's first line. */
+.prb-doc .pr-helper-finding-ref {
+  flex: none;
+  margin-top: 1px;
+  font-family: ui-monospace, SFMono-Regular, 'SF Mono', Menlo, monospace;
+  font-size: var(--t-micro);
+  line-height: 18px;
+  color: var(--pane-ink-muted);
+}
+
 /* ── Feedback selection ──
    Choosing the Issues and Questions to raise with the author. The checkboxes ship on every card
    and show only while the body carries the mode, so entering it is a stylesheet change, as the

package/static/sidebar.css

@@ -5474,6 +5474,21 @@
 .pr-helper-reassessment-evidence .pr-helper-issue-loc { margin-top: 2px; }
 /* Muted, not alarming: an unverified suggestion is still a reasoned one, and can still be accepted. */
 .pr-helper-reassessment-unverified { margin: 2px 0 0; font-size: 11px; font-style: italic; color: #9a6700; }
+/* A status accepted from later evidence, or suggested: the answer as the evidence gives it, then the
+   words it rests on, quoted as pasted, ruled like a comment's evidence because it is someone's claim. */
+.pr-helper-reassessment-answer { margin: 2px 0 0; font-size: 12px; line-height: 1.5; }
+.pr-helper-reassessment-quote {
+  margin: 4px 0 0;
+  padding: 2px 0 2px 8px;
+  border-left: 2px solid rgba(15, 23, 42, 0.18);
+  font-size: 11px;
+  line-height: 1.5;
+  color: var(--color-fg-muted, #57606a);
+  white-space: pre-wrap;
+  overflow-wrap: anywhere;
+}
+.pr-helper-reassessment-basis { margin: 4px 0 0; font-size: 11px; color: var(--color-fg-muted, #57606a); }
+.pr-helper-reassessment-contradicts { margin: 2px 0 0; font-size: 11px; font-style: italic; color: #9a6700; }
 /* A finding's status history reuses the grade history's disclosure and steps. The quote is the
    evidence as it read when the decision was made, which may no longer be what the thread says. */
 .pr-helper-status-step-headline { font-weight: 600; }