docs(ui): note the symmetric Immersive button in FullscreenLyrics header#126
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughMise à jour de la documentation UI/UX pour décrire l'overlay Immersive Now Playing : points d'entrée depuis PlayerBar (clic cover et bouton Maximize2), navigation bidirectionnelle vers Lyrics fullscreen, fermeture via Escape/X, et gestion locale de l'état sans PlayerContext. ChangesDocumentation Immersive Now Playing
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
docs/features/ui.mdlisted the PlayerBar entry points but never mentioned the new Maximize2 button in the FullscreenLyrics header that closes the round-trip Lyrics → Immersive (added in PR feat(lyrics): add immersive button to fullscreen lyrics header #124 / issue feat: add a Immersive Now Playing button from Fullscreen Lyrics #109).Test plan
Summary by CodeRabbit