Fix #334: label picker row grid #342
No reviewers
Labels
No labels
actions
bug
cli
duplicate
enhancement
fork
forum
git storage
help wanted
insights
invalid
issues
moderation
oidc
ownership transfer
packages
pr/merge protection rules
projects
pull requests
question
releases
sponsorships
tags
webhooks
wiki
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
crueber/walhub!342
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/issue-334"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fixes #334 — label picker rows misaligned/wrapped with multi-word pack names + long descriptions.
Change (
web/src/components/LabelPicker.jsx): each row was one flat flex line (name wrapped freely, description truncated with no constrained flex context). Rows are now a three-column gridgrid-cols-[auto_minmax(0,1fr)_minmax(0,1fr)]— [check+dot grouped shrink-0] [namewhitespace-nowrap+ ellipsis] [descriptiontruncate min-w-0] — so every row aligns; rowtitlecarries the full name + description. Panel widensw-64→w-80(320px, inside the #278100vw-1rembound — no 390px shrink needed) with thelabel-drop cardhook untouched (opaque + viewport-bound, no new CSS, no new deps).Preserved: menuitemcheckbox, aria-checked, busy-disable, outside-click/Esc close, native-button keyboard (Tab/Enter/Space free), dark+light (no color change).
MilestonePicker finding (per issue request): checked — its rows are a single text span with no description column, so wrapped titles cannot ragged-shift a sibling column. Left untouched; noted in the doc decision.
Tests: new
web/test/unit/label-picker-rows.test.js(grid anatomy, nowrap/truncate, title, semantics, #278 no-regression, 390px arithmetic). Full web unit suite: 658 pass, 0 fail (node --test web/test/unit/*.test.js).vite build+ esbuild SDK green; verified the arbitrary grid-cols rule is generated into the shipped CSS. No Go changes (make vet/make coverunaffected).Browser open: no live-browser proof — the shared obscura daemon blocks loopback targets and workspace rules forbid a private daemon. Visual check (both themes, 390px, zero console errors) left for review.