docs(quick-006): complete remove list icon from playlist names plan
- Add 6-SUMMARY.md with execution results - Update STATE.md with quick task 006 completion
This commit is contained in:
+5
-3
@@ -82,14 +82,15 @@ None currently.
|
||||
| 003 | Add multi-select to playlist view with context menu delete support | 2026-02-28 | c92ced2 | [3-add-multi-select-to-playlist-view-with-c](./quick/3-add-multi-select-to-playlist-view-with-c/) |
|
||||
| 004 | Add "set as default playlist" context menu option for single playlist selection | 2026-02-28 | 9971b63 | [4-add-set-as-default-playlist-context-menu](./quick/4-add-set-as-default-playlist-context-menu/) |
|
||||
| 005 | Add sort dropdown to playlist view | 2026-03-01 | 5c07485 | [5-add-sort-dropdown-to-playlist-view](./quick/5-add-sort-dropdown-to-playlist-view/) |
|
||||
| 006 | Remove list icon from playlist names, add favorites icon to default | 2026-03-01 | 3c19766 | [6-remove-list-icon-from-playlist-names-and](./quick/6-remove-list-icon-from-playlist-names-and/) |
|
||||
|
||||
## Session Continuity
|
||||
|
||||
### Last Session
|
||||
|
||||
**Date:** 2026-03-01
|
||||
**What happened:** Executed quick task 005 — added sort dropdown to playlist view with four sort options and direction toggle
|
||||
**Where we stopped:** Completed quick task 005 (all tasks, verification passed)
|
||||
**What happened:** Executed quick task 006 — removed list icon from playlist names, added favorites icon (heart/star) to default playlist only
|
||||
**Where we stopped:** Completed quick task 006 (all tasks, verification passed)
|
||||
**Next action:** `/gsd-plan-phase 2` to create execution plan for Backend Correctness
|
||||
|
||||
### Context for Next Session
|
||||
@@ -99,8 +100,9 @@ None currently.
|
||||
- Library and Playlist gained struct-level mutexes; Queue and Player already had them
|
||||
- Ready for Phase 2 (Backend Correctness) — error handling, config permissions, MPRIS errors
|
||||
- Quick task 005: Playlist view now has sort dropdown (Recent, Name, Date Created, Track Count) with persistent preferences
|
||||
- Quick task 006: Playlist list icon removed; default playlist shows favorites icon (heart/star per config), others show no icon
|
||||
|
||||
---
|
||||
*State initialized: 2026-02-27*
|
||||
Last activity: 2026-03-01 - Completed quick task 005: Add sort dropdown to playlist view
|
||||
Last activity: 2026-03-01 - Completed quick task 006: Remove list icon from playlist names, add favorites icon to default
|
||||
*Last updated: 2026-03-01*
|
||||
|
||||
@@ -0,0 +1,103 @@
|
||||
---
|
||||
phase: quick-006
|
||||
plan: 1
|
||||
type: execute
|
||||
wave: 1
|
||||
depends_on: []
|
||||
files_modified:
|
||||
- frontend/src/components/playlist-view/playlist-view.ts
|
||||
autonomous: true
|
||||
requirements: [QUICK-006]
|
||||
|
||||
must_haves:
|
||||
truths:
|
||||
- "Playlist entries in the playlist list do NOT show a 'list' icon before the name"
|
||||
- "The default (favorites) playlist entry shows a heart or star icon (matching favoritesStore iconStyle) instead of no icon"
|
||||
- "Non-default playlists show no icon between the chevron and the name"
|
||||
artifacts:
|
||||
- path: "frontend/src/components/playlist-view/playlist-view.ts"
|
||||
provides: "Updated playlist item rendering without list icon, with favorites icon for default playlist"
|
||||
key_links:
|
||||
- from: "renderPlaylistItem"
|
||||
to: "favCtrl.playlistId / favCtrl.iconName"
|
||||
via: "Conditional icon rendering based on default playlist ID match"
|
||||
pattern: "favCtrl\\.playlistId|favCtrl\\.iconName"
|
||||
---
|
||||
|
||||
<objective>
|
||||
Remove the "list" icon that appears before every playlist name in the playlist view, and add the user-configured favorites icon (heart or star) to the default playlist entry only.
|
||||
|
||||
Purpose: Cleaner playlist list — the list icon adds visual noise; the favorites icon on the default playlist gives quick visual identification.
|
||||
Output: Updated playlist-view.ts with conditional icon rendering.
|
||||
</objective>
|
||||
|
||||
<execution_context>
|
||||
@.planning/quick/6-remove-list-icon-from-playlist-names-and/6-PLAN.md
|
||||
</execution_context>
|
||||
|
||||
<context>
|
||||
@frontend/src/components/playlist-view/playlist-view.ts (main file to modify)
|
||||
@frontend/src/store/controllers/favorites-controller.ts (provides favCtrl.playlistId, favCtrl.iconName)
|
||||
|
||||
<interfaces>
|
||||
From frontend/wailsjs/go/models.ts (playlist namespace):
|
||||
```typescript
|
||||
export class Summary {
|
||||
ID: number;
|
||||
Name: string;
|
||||
CreatedAt: string;
|
||||
UpdatedAt: string;
|
||||
}
|
||||
```
|
||||
|
||||
From frontend/src/store/controllers/favorites-controller.ts:
|
||||
```typescript
|
||||
// Already instantiated on the component as: private favCtrl = new FavoritesController(this);
|
||||
get playlistId(): number; // Returns the default playlist's DB ID
|
||||
get iconName(): string; // Returns 'star' or 'heart' based on user config
|
||||
```
|
||||
</interfaces>
|
||||
</context>
|
||||
|
||||
<tasks>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 1: Remove list icon from all playlists and add favorites icon to default playlist</name>
|
||||
<files>frontend/src/components/playlist-view/playlist-view.ts</files>
|
||||
<action>
|
||||
In the `renderPlaylistItem` method (~line 2883), replace the static `<wa-icon class="playlist-icon" name="list"></wa-icon>` block (lines 2923-2926) with a conditional:
|
||||
|
||||
- If `entry.summary.ID === this.favCtrl.playlistId`, render `<wa-icon class="playlist-icon" name=${this.favCtrl.iconName}></wa-icon>` (shows heart or star per user config)
|
||||
- Otherwise, render nothing (no icon at all between chevron and name)
|
||||
|
||||
The existing `.playlist-icon` CSS class (lines 568-572) should remain — it styles the icon for the default playlist entry. No CSS changes needed.
|
||||
|
||||
Also update the `.playlist-body` left padding from `42px` to `32px` (line 590) to tighten the track list indentation now that most rows no longer have the icon taking up ~28px (18px icon + 10px gap). This keeps the tracks visually aligned under the playlist name rather than indented too far.
|
||||
|
||||
Do NOT touch the empty-state `<wa-icon name="list">` on line 2818 — that's the "no playlists" illustration, not a per-playlist icon.
|
||||
</action>
|
||||
<verify>
|
||||
npm run --prefix frontend check (TypeScript compiles without errors)
|
||||
</verify>
|
||||
<done>
|
||||
- No playlist entry shows the "list" icon
|
||||
- The default/favorites playlist entry shows the heart or star icon (matching user config)
|
||||
- Non-default playlists show only the chevron then the name (no icon between)
|
||||
- TypeScript compiles cleanly
|
||||
</done>
|
||||
</task>
|
||||
|
||||
</tasks>
|
||||
|
||||
<verification>
|
||||
- `npm run --prefix frontend check` passes
|
||||
- Visual: In the playlist view, non-default playlists show chevron → name (no icon). The default playlist shows chevron → heart/star → name.
|
||||
</verification>
|
||||
|
||||
<success_criteria>
|
||||
The list icon is removed from all playlist entries. The default playlist entry displays the user-configured favorites icon (heart or star). All other playlists show no icon. TypeScript compiles without errors.
|
||||
</success_criteria>
|
||||
|
||||
<output>
|
||||
After completion, create `.planning/quick/6-remove-list-icon-from-playlist-names-and/6-SUMMARY.md`
|
||||
</output>
|
||||
@@ -0,0 +1,50 @@
|
||||
---
|
||||
phase: quick-006
|
||||
plan: 1
|
||||
subsystem: frontend
|
||||
tags: [ui, playlist, icons]
|
||||
dependency_graph:
|
||||
requires: [favorites-controller]
|
||||
provides: [conditional-playlist-icons]
|
||||
affects: [playlist-view]
|
||||
tech_stack:
|
||||
patterns: [conditional-lit-rendering, nothing-sentinel]
|
||||
key_files:
|
||||
modified:
|
||||
- frontend/src/components/playlist-view/playlist-view.ts
|
||||
decisions:
|
||||
- Used `nothing` from lit instead of empty string for clean DOM when no icon needed
|
||||
metrics:
|
||||
duration: 1 min
|
||||
completed: "2026-03-01T14:44:26Z"
|
||||
---
|
||||
|
||||
# Quick Task 6: Remove List Icon from Playlist Names and Add Favorites Icon
|
||||
|
||||
Conditional icon rendering in playlist list — favorites icon (heart/star per user config) on default playlist, no icon on others, tighter body padding.
|
||||
|
||||
## What Changed
|
||||
|
||||
### Task 1: Remove list icon, add conditional favorites icon
|
||||
**Commit:** `3c19766`
|
||||
|
||||
- **Removed** the static `<wa-icon name="list">` that appeared before every playlist name
|
||||
- **Added** conditional rendering: if `entry.summary.ID === this.favCtrl.playlistId`, renders the user-configured favorites icon (`heart` or `star`); otherwise renders `nothing` (no DOM element)
|
||||
- **Reduced** `.playlist-body` left padding from `42px` to `32px` to tighten track list indentation now that most rows lack the icon
|
||||
- The empty-state `<wa-icon name="list">` (line 2818) was intentionally left untouched
|
||||
|
||||
## Deviations from Plan
|
||||
|
||||
None — plan executed exactly as written.
|
||||
|
||||
## Verification
|
||||
|
||||
- `vite build` compiled 283 modules successfully
|
||||
- Pre-commit hook `frontend-typecheck` passed
|
||||
- Default playlist shows favorites icon (heart/star per user config)
|
||||
- Non-default playlists show chevron directly followed by name (no icon)
|
||||
|
||||
## Self-Check: PASSED
|
||||
|
||||
- [x] `frontend/src/components/playlist-view/playlist-view.ts` exists
|
||||
- [x] Commit `3c19766` exists in git history
|
||||
Reference in New Issue
Block a user