From 9f7dd1a7e91ed2ea9ca1eaaed3b65a7a68880fdb Mon Sep 17 00:00:00 2001 From: amosblomqvist Date: Thu, 24 Sep 2026 15:51:40 +0300 Subject: [PATCH] Refinement round: ring popover down, menu order, tok unit, list indent, lowercase levels MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Five small fixes (owner refinement round, 2026-09-24): 1. RING POPOVER OPENS DOWN: the ring lives in the chat top bar — the window's top edge is right above it, so the upward popover was clipped. top-full mt-1.5 now: it drops below the ring. 2. SESSION MENU ORDER FLIPPED: Rename/Delete first, hairline, then the extraction coverage block + Extract at the bottom — the info reads after the actions (was the reverse, the block on top). 3. UNIT TYPED OUT: the sidebar block reads "2k tok uncovered" now — the tokens unit is explicit (owner: "tok"). 4. MARKDOWN LIST INDENT: streamdown ships list-inside with no container padding — top-level markers sat ON the text edge (only nested lists got their pl-6 via [li_&]). .lh-prose :where(ul, ol) { padding-left: 1.5rem } indents both list kinds as blocks, 24px — the same measure as the nested-case pl-6. Unlayered CSS beats the utility layer; :where keeps specificity zero. 5. THINKING LEVELS LOWERCASE EVERYWHERE: the composer pill and setup dialog rendered THINKING_LABEL (Off/Minimal/…/Max) while every menu body renders the raw lowercase level — the map is now lowercase, the level string IS the display form, everywhere. Live-verified on :3001: list content inset 24px vs paragraphs (padding-left computed 24px, nested fine); the composer pill reads "low" with options low/high/max; menu order Rename → Delete… → hairline → block. 901/901. --- app/globals.css | 10 +++++++++ components/extraction-coverage.tsx | 6 +++--- components/sidebar.tsx | 26 +++++++++++------------ components/thinking-default.ts | 20 ++++++++++------- tests/render-extraction-coverage.test.tsx | 2 +- 5 files changed, 39 insertions(+), 25 deletions(-) diff --git a/app/globals.css b/app/globals.css index efb8a29..c1c9c7c 100644 --- a/app/globals.css +++ b/app/globals.css @@ -197,6 +197,16 @@ body { color: var(--lh-ink-italic); } +/* Markdown lists indent from the prose edge (owner ask, 2026-09-24): + streamdown ships list-inside with no container padding (top-level + markers sit ON the text edge — nested lists get their pl-6 only via + [li_&]). 24px — the same measure as that nested-case pl-6 — pulls + the whole list right as a block. Unlayered CSS beats the utility + layer; :where keeps specificity zero. */ +.lh-prose :where(ul, ol) { + padding-left: 1.5rem; +} + /* Markdown headings render at the font's BLACK weight (owner ask, 2026-09-19) — 900, not streamdown's font-semibold 600. The variable Source Sans 3 axis spans 200–900 (next/font loads the full range), so diff --git a/components/extraction-coverage.tsx b/components/extraction-coverage.tsx index 9758c2b..bf572d9 100644 --- a/components/extraction-coverage.tsx +++ b/components/extraction-coverage.tsx @@ -97,7 +97,7 @@ async function requestExtract(chatId: string): Promise { // ─── The sidebar context menu's coverage block ────────────────────────────── /** - * The info block at the top of the session context menu (owner-locked + * The info block at the bottom of the session context menu (owner-locked * redesign): what the extractor hasn't covered yet, plus the manual Extract * affordance — visually separated from Rename/Delete below (those stay the * two session-management actions). Fetched ON MENU OPEN (the block mounts @@ -191,7 +191,7 @@ export function CoverageMenuBody({ ) : ( - {formatTokens(coverage.uncoveredTokens)} uncovered + {formatTokens(coverage.uncoveredTokens)} tok uncovered {coverage.uncoveredCount} message{coverage.uncoveredCount === 1 ? "" : "s"} unextracted @@ -305,7 +305,7 @@ export function CoverageRing({ chatId, refreshKey = 0 }: { chatId: string; refre {open && (
diff --git a/components/sidebar.tsx b/components/sidebar.tsx index 0f266d9..0809bc0 100644 --- a/components/sidebar.tsx +++ b/components/sidebar.tsx @@ -377,11 +377,11 @@ function SessionsSidebar() { const listRef = useRef(null); /* ── Wave 6 chunk A: context menu + inline rename + deletion ── - Right-click a session row → MenuShell: the extraction coverage - block (owner-locked redesign, 2026-09-24 — numbers + the manual - Extract affordance, separated by a hairline) above the two - session-management actions (owner ruling): Rename (inline, below) - and Delete (the two-option dialog). The anchor is the row that was + Right-click a session row → MenuShell: the two management actions + first (owner ruling), then — below a hairline — the extraction + coverage block (owner-locked redesign 2026-09-24, order refined + 2026-09-24: Rename/Delete on top, the numbers + manual Extract + affordance at the bottom). The anchor is the row that was right-clicked — assigned into a ref before the state flip so the shell measures it on its first (hidden) pass. */ const [ctxChatId, setCtxChatId] = useState(null); @@ -662,14 +662,12 @@ function SessionsSidebar() { preferredSide="down" menuClass="flex w-52 flex-col gap-0.5 rounded-xl border bg-card p-2 text-xs shadow-xl" > - {/* The extraction block (owner-locked redesign, 2026-09-24): - what the extractor hasn't covered yet + the manual Extract - affordance — visually separated from the management - actions below by a hairline. Fetched on menu open (the - block mounts WITH the menu); 409 = benign (already - running). Renders nothing on a failed read. */} - -