Skip to content

feat: show album art ring only on hover in listening card#54

Merged
kaiiiichen merged 1 commit intomainfrom
feat/listening-card-hover-ring
May 8, 2026
Merged

feat: show album art ring only on hover in listening card#54
kaiiiichen merged 1 commit intomainfrom
feat/listening-card-hover-ring

Conversation

@kaiiiichen
Copy link
Copy Markdown
Owner

Summary

  • Move the 1.5px orange ring around the album cover behind a group-hover/track: selector so it only appears while hovering the track row, instead of being permanently visible whenever a track is "now playing".
  • Apply the same hover-ring treatment in both the now playing and last played states (and the no-art placeholder fallback) so the hover affordance is consistent.
  • Add box-shadow to the album-art transition list so the ring fades in/out smoothly.

The breathing listening-art-live animation is unchanged — only the static ring moved to hover.

Test plan

  • npm run lint
  • npm run typecheck
  • npm run test
  • npm run build
  • Visual: hover the album art in the listening card while a track is playing — ring fades in. Move cursor away — ring fades out.
  • Visual: same behavior in "last played" state.
  • Visual: same behavior in dark mode (uses #D9A870 ring + #252019 offset).

Made with Cursor

The 1.5px orange ring around the album cover was always visible whenever
a track was "now playing" — it pulled too much attention from the rest
of the card. Move it behind a `group-hover/track:` selector so it only
appears while hovering the track row, and apply it uniformly for both
the "now playing" and "last played" states (and the no-art placeholder
fallback) so the hover affordance is consistent.

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
kaichen.dev Ready Ready Preview, Comment May 8, 2026 9:37am

@kaiiiichen kaiiiichen merged commit 8589b09 into main May 8, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant