Skip to content

fix(ui): stack edit drawer — labeled multi-line slot cards + fix escaped toggle knob#1023

Merged
thinmintdev merged 1 commit into
mainfrom
fix/stack-drawer-ui
Jul 4, 2026
Merged

fix(ui): stack edit drawer — labeled multi-line slot cards + fix escaped toggle knob#1023
thinmintdev merged 1 commit into
mainfrom
fix/stack-drawer-ui

Conversation

@thinmintdev

Copy link
Copy Markdown
Contributor

Stack edit drawer UI — decrammed, labeled, dots fixed

Viewed the current implementation live on lxc105 (Playwright against the running deployment), diagnosed the exact issues, and redesigned.

Problems (measured from the live DOM)

  • Cramped, unlabeled fields: the per-slot editor packed 6 controls into one grid row 110px 1fr 96px 1fr auto auto, squeezing the model and profile selects to ~45px each — they only showed —▾, unreadable — while short "ROCm" got 96px. No field labels.
  • Stray indicator dots overlapping the panel: .pf-switch had no position: relative, so the absolutely-positioned .pf-switch-knob escaped to x=324 (the drawer's left border) instead of sitting on its track.

Fix

  • Each slot is now a labeled card: a header row (full-width slot name + remove) above a two-column labeled field gridModel spans full width (long ids), with Device / Profile / Speculative decode labeled beneath. Every field is fully visible and labeled; wraps to one column under 720px.
  • Added position: relative to .pf-switch → the knob sits on its track (fixes the toggle everywhere, not just here).

Verified (Playwright vs live lxc105 data via vite dev proxy)

  • Model select width 45px → 395px; both toggle knobs insideSwitch: true (offset 3px on track).
  • The one remaining left-edge dot is the dimmed active-stack hero behind the scrim (page content, elementAtDot = pf-scrim), not a drawer artifact.
  • npm run build clean; all data-testids preserved. (No pre-existing stack-drawer e2e — CI γ-suite covers global regressions.)

🤖 Generated with Claude Code

…ped toggle knob

The per-slot editor crammed 6 controls into one grid row
(110px 1fr 96px 1fr auto auto), squeezing the model and profile selects to
~45px each (unreadable, just '—▾') with no labels. And .pf-switch lacked
position:relative, so the absolutely-positioned .pf-switch-knob escaped to
the drawer's left edge (the stray grey 'indicator dots' overlapping the panel).

Redesign each slot as a labeled card: a header row (full-width slot name +
remove) above a two-column labeled field grid — Model spans full width, with
Device / Profile / Speculative-decode labeled beneath. Every field is now
fully visible and labeled. Add position:relative to .pf-switch so the knob
sits on its track (fixes it everywhere). data-testids preserved.

Verified live against lxc105 data via the vite dev proxy + Playwright:
model select 395px (was 45px), knobs inside their switches.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@thinmintdev thinmintdev enabled auto-merge (squash) July 4, 2026 05:24
@thinmintdev thinmintdev merged commit ab9ffc9 into main Jul 4, 2026
3 checks passed
thinmintdev added a commit that referenced this pull request Jul 4, 2026
…ce + logs/memory fixes

Bumps version 0.8.3b1 → 0.8.4b1 and folds the [Unreleased] CHANGELOG into the
v0.8.4b1 section covering everything since the v0.8.3b1 tag: model preferred
profiles, ComfyUI/image-gen model surface, persistent slot context, unified
logs/events, memory audit trail + console shape guards (#1024/#1026/#1028/#1030),
stack edit drawer (#1023), board kanban drawer (#1014), and optional-cosign
installer.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant