prreviewbuddy 0.48.0 → 0.49.0

package/static/shell.css

@@ -806,6 +806,8 @@ a.prb-author:hover .prb-author-name { color: var(--text-primary); text-decoratio
 
 /* ── Three-pane body ── */
 .prb-body {
+  /* Positioned, for Buddy AI's edge tab, which hangs off this box's right edge at its top. */
+  position: relative;
   display: grid;
   grid-template-columns: var(--rail-width) minmax(0, 1fr) var(--ask-width);
   /* Whatever the banners above did not take, and `0` rather than `auto` as the basis: the height
@@ -1887,17 +1889,6 @@ a.prb-rail-item { text-decoration: none; }
   font-size: var(--t-meta);
 }
 
-.prb-ask-toggle {
-  padding: 6px 11px;
-  border-radius: 7px;
-  border: 1px solid var(--border);
-  background: var(--surface-2);
-  color: var(--text-secondary);
-  font: inherit;
-  font-size: var(--t-meta);
-  cursor: pointer;
-}
-.prb-ask-toggle:hover { color: var(--text-primary); border-color: var(--border-strong); }
 
 
 /* Re-reading the conversation, offered on the PR context view. Light pane: literals, not tokens. */
@@ -2034,21 +2025,6 @@ a.prb-rail-item { text-decoration: none; }
 .prb-fresh-status[data-state='changed'] { color: var(--warning); }
 .prb-fresh-status[data-state='changed']::before { background: var(--warning); }
 .prb-fresh-status[data-state='current']::before { background: var(--success); }
-/* The action beside it is a button like Mark reviewed beside it: the same secondary treatment, so
-   the header reads as one status followed by the things you can do. */
-.prb-fresh-check {
-  padding: 4px 10px;
-  border: 1px solid var(--border);
-  border-radius: 7px;
-  background: transparent;
-  color: var(--text-secondary);
-  font: inherit;
-  font-size: var(--t-meta);
-  white-space: nowrap;
-  cursor: pointer;
-}
-.prb-fresh-check:hover { border-color: var(--border-strong); color: var(--text-primary); }
-.prb-fresh-check:disabled { cursor: default; color: var(--text-muted); border-color: var(--border); }
 
 /* ── Review freshness ──
  *
@@ -3609,23 +3585,18 @@ a.prb-rail-item { text-decoration: none; }
   line-height: var(--lh-body);
 }
 
-/* Header controls that only exist where there is a server behind them. */
-.prb-complete {
-  padding: 4px 10px;
-  border: 1px solid var(--border);
-  border-radius: 7px;
-  background: transparent;
-  color: var(--text-secondary);
-  font: inherit;
+/* The review page's lifecycle pill is the status pill's shape. Changed since it was parked, it
+   reads as the card's chip does: a sentence, tinted like news. */
+.prb-page-lifecycle[data-changed] {
+  border-color: transparent;
+  background: var(--brand-muted);
+  color: var(--brand-ink);
   font-size: var(--t-meta);
-  cursor: pointer;
-}
-.prb-complete:hover { border-color: var(--border-strong); color: var(--text-primary); }
-.prb-complete[aria-pressed="true"] {
-  border-color: var(--success-border);
-  background: var(--success-subtle);
-  color: var(--success-ink);
+  font-weight: 600;
+  letter-spacing: 0;
+  text-transform: none;
 }
+.prb-page-lifecycle[hidden] { display: none; }
 
 /* ── Diagrams ──
  *
@@ -4998,7 +4969,8 @@ body .prb-body[data-focus='buddy'] .prb-msg-user { max-width: min(90%, 78ch); }
 
 /* The expand control. Icon-only, like Copy on a card, so it does not compete with a title. Both
    icons are in the markup and `aria-pressed` picks one, which is the state the client sets. */
-.prb-focus-toggle {
+.prb-focus-toggle,
+.prb-ask-hide {
   display: inline-flex;
   align-items: center;
   justify-content: center;
@@ -5012,14 +4984,16 @@ body .prb-body[data-focus='buddy'] .prb-msg-user { max-width: min(90%, 78ch); }
   color: var(--text-muted);
   cursor: pointer;
 }
-.prb-focus-toggle:hover { color: var(--text-primary); background: var(--surface-2); }
-.prb-focus-toggle:focus-visible { outline: 2px solid var(--brand-hover); outline-offset: -2px; }
+.prb-focus-toggle:hover,
+.prb-ask-hide:hover { color: var(--text-primary); background: var(--surface-2); }
+.prb-focus-toggle:focus-visible,
+.prb-ask-hide:focus-visible { outline: 2px solid var(--brand-hover); outline-offset: -2px; }
 .prb-focus-toggle .prb-focus-collapse,
 .prb-focus-toggle[aria-pressed='true'] .prb-focus-expand { display: none; }
 .prb-focus-toggle[aria-pressed='true'] .prb-focus-collapse { display: block; }
 /* In the assistant's header it follows the scope label, which takes the free space with
    `margin-left: auto`, so the control stays at the right edge beside it. */
-.prb-ask-header .prb-focus-toggle { align-self: center; }
+.prb-ask-header :is(.prb-focus-toggle, .prb-ask-hide) { align-self: center; }
 
 /* The same control in the light document pane, in the pane's palette. */
 .prb-doc .prb-focus-toggle { color: var(--pane-ink-faint); }
@@ -5151,3 +5125,45 @@ body .prb-body[data-focus='buddy'] .prb-msg-user { max-width: min(90%, 78ch); }
   letter-spacing: 0.04em;
   text-transform: uppercase;
 }
+
+/* ── Buddy AI's edge tab ──
+ * On the body's right edge, at its top, where the pane opens. Against the body rather than the
+ * window, so a banner above the body moves it down with everything else. Only while the pane is not
+ * showing: the pane's own header has the chevron that closes it.
+ *
+ * Here at the end of the file on purpose. The hiding rules have to come after every breakpoint
+ * block that decides whether the pane shows, or a base rule below one of them silently wins. */
+.prb-ask-tab {
+  position: absolute;
+  top: 16px;
+  right: 0;
+  z-index: 4;
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+  padding: 7px 10px 7px 11px;
+  border: 1px solid var(--border);
+  border-right: 0;
+  border-radius: 8px 0 0 8px;
+  background: var(--surface);
+  box-shadow: -2px 2px 8px var(--shadow-soft);
+  color: var(--text-secondary);
+  font: inherit;
+  font-size: var(--t-meta);
+  font-weight: 600;
+  white-space: nowrap;
+  cursor: pointer;
+}
+.prb-ask-tab:hover { color: var(--text-primary); border-color: var(--border-strong); }
+.prb-ask-tab:focus-visible { outline: 2px solid var(--brand-hover); outline-offset: -2px; }
+.prb-ask-tab-icon { flex: none; color: var(--brand); }
+
+/* The pane is showing: chosen open, or wide enough to show by default with no choice made. */
+.prb-body[data-ask='open'] .prb-ask-tab { display: none; }
+@media (min-width: 1181px) {
+  .prb-body:not([data-ask]) .prb-ask-tab { display: none; }
+}
+/* Buddy expanded fills the body; another surface expanded hides the pane, so the tab is the way
+   back to it, whatever `data-ask` says underneath. */
+body .prb-body[data-focus='buddy'] .prb-ask-tab { display: none; }
+body .prb-body[data-focus]:not([data-focus='buddy']) .prb-ask-tab { display: inline-flex; }