bvandeusen/minstrel
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 38 Wiki Activity

Web Home: drop hero row + compact Rediscover tiles #66

Merged
bvandeusen merged 1 commits from dev into main 2026-06-01 20:43:53 -04:00
Conversation 0 Commits 1 Files Changed 3
+10 -171
bvandeusen commented 2026-06-01 20:43:45 -04:00
Owner
Copy Link
Copy Source

Operator UI review on the merged build.

  • Hero row removed. Operator chose "remove entirely" on the question — the Playlists row already leads with For-You and Recently Added already leads with the newest album, so the hero duplicated both anchors. Pulled HomeHeroCard.svelte + all page-level wiring (the systemShuffle/getPlaylist imports + playForYou/playLatestAlbum handlers) and reverted the within() test scoping added in 682d7a5e.
  • Rediscover compact tiles. w-32 for albums / w-28 for artists, matching the Most Played CompactTrackCard visual weight on web. Reinforces the page hierarchy: fresh content gets real estate, throwbacks recede.

CI green on d4371951 (run 153). Single-commit PR.

Operator UI review on the merged build. - **Hero row removed.** Operator chose "remove entirely" on the question — the Playlists row already leads with For-You and Recently Added already leads with the newest album, so the hero duplicated both anchors. Pulled `HomeHeroCard.svelte` + all page-level wiring (the `systemShuffle`/`getPlaylist` imports + `playForYou`/`playLatestAlbum` handlers) and reverted the `within()` test scoping added in `682d7a5e`. - **Rediscover compact tiles.** `w-32` for albums / `w-28` for artists, matching the Most Played CompactTrackCard visual weight on web. Reinforces the page hierarchy: fresh content gets real estate, throwbacks recede. CI green on `d4371951` (run 153). Single-commit PR.
bvandeusen added 1 commit 2026-06-01 20:43:45 -04:00
feat(web): drop hero row + compact Rediscover tiles
test-web / test (push) Successful in 33s
Details
d43719516e 
Operator UI review on the merged build:

- Hero row felt 'odd' — operator chose remove-entirely on the
  AskUserQuestion options. The Playlists row already leads with
  For-You and Recently Added leads with the newest album, so the
  hero duplicated both anchors. Pulled HomeHeroCard.svelte + all
  page-level wiring (systemShuffle/getPlaylist imports + playForYou/
  playLatestAlbum handlers) and reverted the within() test scoping
  added in 682d7a5e.

- Rediscover tiles step down to w-32 (albums) / w-28 (artists),
  matching the Most Played CompactTrackCard visual weight on web.
  Reinforces the page hierarchy: fresh content gets real estate,
  throwbacks recede.
bvandeusen merged commit 7de238e91e into main 2026-06-01 20:43:53 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
deps
Dependency update (Renovate)
 deps:major
Major dependency bump — held for manual review
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
bvandeusen
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bvandeusen/minstrel#66
Delete Branch "dev"

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?