Web UI: Most Played hover fix, narrower seek bar, Android-parity track kebab #99

Merged

bvandeusen merged 5 commits from dev into main 2026-06-14 22:26:52 -04:00

Conversation 0 Commits 5 Files Changed 12

+88 -390

5 Commits

Author	SHA1	Message	Date
bvandeusenandClaude Opus 4.8	f7278f2417	feat(web): drop "Remove from library" from the track kebab ... No single-click destructive action belongs in the kebab. Removing the item orphaned its whole path (RemoveTrackPopover was its only caller, and the admin/tracks API client was the popover's only caller), so per the repo's no-dead-code convention the chain is fully removed: the menu item + its admin/isAdmin plumbing in TrackMenu, RemoveTrackPopover(.svelte/.test), src/lib/api/admin/tracks(.ts/.test), and the now-needless transitive mocks in the CompactTrackCard / PlaylistTrackRow / playlist specs. The kebab is now an 8-item, admin-agnostic menu. The DELETE /api/admin/tracks server endpoint is untouched — a future safer admin surface can rebind it. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-14 22:17:35 -04:00
bvandeusenandClaude Opus 4.8	9a166f9032	test(web): update TrackMenu spec for Android-parity kebab ... The kebab gained "Start radio" and dropped the duplicate "Flag this track…" (its action now lives solely under "Hide", which opens the same FlagPopover). Net item count is unchanged (9 admin / 8 non-admin), but the named-item and flag-entry assertions needed updating: - mock playRadio in the store mock; assert Start radio dispatches playRadio. - swap the flag-item presence check for start-radio. - replace the "click Flag" test with "click Hide opens the popover". Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-14 21:48:57 -04:00
bvandeusenandClaude Opus 4.8	b3d6785543	feat(web): match track kebab to Android + add it to full-screen Now Playing ... Bring the web TrackMenu to parity with Android's canonical TrackActionsSheet so the kebab reads the same on both clients: - Reorder to Android's groups: queue → like/add-to-playlist/start-radio → go-to-album/artist → hide. - Drop the duplicate "Flag this track…" item — it opened the very same FlagPopover as "Hide" (Android folds flag into a single Hide). - Align icons (ListVideo / ListMusic / ListPlus / Disc3 / User). - Admin-only "Remove from library" stays as a web superset (Android has no surface for it), past its own divider. Mount the kebab on the full-screen /now-playing route with hideQueueActions, mirroring Android's NowPlayingScreen — Start radio / Add to playlist / Hide were previously unreachable there (only like + volume + queue existed). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-14 21:46:54 -04:00
bvandeusenandClaude Opus 4.8	773275e916	feat(web): surface "Start radio" in the track kebab menu ... Radio was fully wired (playRadio → /api/radio + 80% auto-refresh) but its only entry point was TrackRow's inline 📻 button, so it was unreachable from the kebab — i.e. missing on the Most Played compact cards and the mini-player. Add a "Start radio" item to TrackMenu, shown even under hideQueueActions since reseeding a station from the current track is meaningful there. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-14 21:40:16 -04:00
bvandeusenandClaude Opus 4.8	066616e196	fix(web): de-overlap Most Played hover controls + narrow desktop seek bar ... CompactTrackCard is a short one-line row but reused CardActionCluster, which corner-splits Like+Add (top) and the kebab menu (bottom). That split is right for the tall square Album/Artist cards but makes the two groups collide on the compact row's hover state. Give the compact card a single inline, vertically-centred right cluster (Like + Add + menu in one group) and widen its right padding reserve to match. In the desktop PlayerBar, the left info column was a fixed w-72 (title kept truncating) while the seek column was flex-1 (the scrubber hogged the slack on wide screens). Let the left column grow up to max-w-md while holding its 288px floor at md, and cap the seek/transport column at max-w-xl centred, so freed width flows to the title instead of stretching the bar. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-14 21:36:26 -04:00