fix(ui): music player starts collapsed with a labeled radio pill - #2431
Conversation
Expanded by default, the floating transport strip sat over interactive content (the OTHER MARKETS table header on /trade, the table edge on /markets) and nothing anywhere said what it was — an unlabeled play/volume strip reads as broken UI. It now starts as one small music-note pill labeled "Lo-fi radio" (title + aria), expands on click, and remembers the expanded choice in localStorage (restored in an effect to avoid an SSR hydration mismatch). Audio keeps playing while collapsed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Item 8 from the live-site UX sweep. The floating player overlapped interactive content (OTHER MARKETS header on /trade, table edge on /markets) and was unlabeled — a mystery transport strip. Now: starts as one 32px music-note pill with
title/aria-label"Lo-fi radio", expands on click, choice persisted (effect-restored to dodge hydration mismatch), audio unaffected by collapse. Existing GH#1662 overlap tests: 7/7 green; tsc clean.🤖 Generated with Claude Code