vinzenz/hyperhive
vinzenz/hyperhive
Watch
0
0
Fork
You've already forked hyperhive
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

dashboard: drop stale data-spw note in side-panel-drawer comment (#451)

Browse source 
argus nit on #456: the CSS comment claimed "The body adds
data-spw=<px> once persisted" — that's leftover from a prior
design and no such attribute is ever set. The actual flow is
localStorage → JS reads on Panel.bind → sets --side-panel-w via
inline style. Replaced the misleading line with an accurate
description of the persistence path.
This commit is contained in:
iris 2026-05-26 01:26:51 +02:00 • committed by Mara
commit 002419ba4d
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
frontend/packages/dashboard/src/dashboard.css
View file

@ -1302,10 +1302,10 @@ footer .banner-thin {
/* #451: width is a CSS variable so the drag handle (added by /* #451: width is a CSS variable so the drag handle (added by
Panel.bind) can update it live, and so localStorage-persisted Panel.bind) can update it live, and so localStorage-persisted
widths apply on first paint. Default min(760px, 94vw) preserves widths apply on first paint. Default min(760px, 94vw) preserves
the pre-#451 behaviour for operators who never drag. The body the pre-#451 behaviour for operators who never drag. JS sets
adds `data-spw="<px>"` once persisted; CSS doesn't read it `--side-panel-w` via inline style on the drawer; persistence
directly — JS sets `--side-panel-w` via inline style on the lives in localStorage (key `hyperhive:side-panel-width`),
drawer. */ replayed onto the var by `Panel.applyStoredWidth` at bind. */
width: var(--side-panel-w, min(760px, 94vw)); width: var(--side-panel-w, min(760px, 94vw));
/* Clamp so a stored width can never push the drawer off-screen /* Clamp so a stored width can never push the drawer off-screen
or shrink it past readability. min content width matches the or shrink it past readability. min content width matches the