yonlu/yellowjacket
1
1
Fork 0
Code Issues 55 Pull Requests 6 Actions Packages Projects Releases 12 Wiki Activity

Desktop: make the sidebar's Playlists entry expandable, and a drop target #20

New Issue
Open
opened 2026-08-18 05:54:00 +00:00 by logan · 0 comments
logan commented 2026-08-18 05:54:00 +00:00
Collaborator
Copy Link
Copy Source

Report

The Playlists item in the sidebar should expand to list my playlists, for quick access — and I should be able to drag tracks/albums onto a playlist name there to add them to it.

Findings

  • frontend/src/components/sidebar/app-sidebar.ts has a flat item list (~lines 199-209); no nesting.
  • playlist-store is deliberately lazy: playlist-view is its only reader and is created lazily, so neither the invalidation nor the singleton's construction warms a cache for a page that may never open. An always-mounted sidebar list changes that — the sidebar becomes a permanent subscriber.
  • PlaylistTracksChanged already carries the playlist id and patches in place, so the cost of subscribing is bounded.
  • The sidebar collapses to icon mode below 900px and is reused inside the mobile drawer — the expanded list has to behave in both.

Direction

A disclosure on the Playlists row (aria-expanded/aria-controls, body rendered and hidden-toggled — see config-section's pattern) plus drop handling per playlist row reusing the existing drag payload. Decide explicitly whether the sidebar warming playlist-store at startup is acceptable, or whether the list should load on first expand.

**Report** The Playlists item in the sidebar should expand to list my playlists, for quick access — and I should be able to drag tracks/albums onto a playlist name there to add them to it. **Findings** - `frontend/src/components/sidebar/app-sidebar.ts` has a flat item list (~lines 199-209); no nesting. - `playlist-store` is deliberately lazy: `playlist-view` is its only reader and is created lazily, so neither the invalidation nor the singleton's construction warms a cache for a page that may never open. An always-mounted sidebar list changes that — the sidebar becomes a permanent subscriber. - `PlaylistTracksChanged` already carries the playlist id and patches in place, so the cost of subscribing is bounded. - The sidebar collapses to icon mode below 900px and is reused inside the mobile drawer — the expanded list has to behave in both. **Direction** A disclosure on the Playlists row (`aria-expanded`/`aria-controls`, body rendered and `hidden`-toggled — see `config-section`'s pattern) plus drop handling per playlist row reusing the existing drag payload. Decide explicitly whether the sidebar warming `playlist-store` at startup is acceptable, or whether the list should load on first expand.
logan added the Area/Shell-NavKind/FeaturePlatform/Desktop
Priority
Medium
3
 labels 2026-08-18 14:36:07 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Area/Design
Area/Downloads
Area/Explore
Area/Library-UI
Area/Metadata
Area/Packaging
Area/Player
Area/Queue
Area/Settings
Area/Shell-Nav
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
 Platform/Android
Platform/Desktop
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Medium
3
The priority is medium
Priority
Low
4
The priority is low
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
In Progress
Somebody is actively working on this right now
No labels Area/Shell-Nav Kind/Feature Platform/Desktop
Priority
Medium
3
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
logan yonlu
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: yonlu/yellowjacket#20