docs(canvas): document the per-track Canvas feature (#442) - #469
Merged
Conversation
Catch up the docs debt for the merged track Canvas (PR #467): - docs/features/ui.md: new "Track Canvas" section — sourcing (manual mp4, hash-addressed + atomic hard_link publish via the shared media_file helper), the Show Canvas toggle (default OFF), and the useTrackCanvas lookup guards (id/profileId tag + profile-generation token). - CLAUDE.md: Canvas clause in the UI feature catalogue. - docs/README.md: mention immersive view + track Canvas in the UI & UX row.
|
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 (3)
📝 WalkthroughWalkthroughLa documentation ajoute Track Canvas au périmètre UI. Elle décrit les vidéos MP4 par piste, leur import et stockage, les commandes de gestion, les priorités d’affichage, l’activation par défaut, les exclusions et les protections contre les résultats obsolètes. ChangesDocumentation Track Canvas
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
This was referenced Aug 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Catches up the documentation debt for the track Canvas feature merged in #467.
## Track Canvassection: sourcing (manual.mp4, magic-byte + size validation, hash-addressed + atomichard_linkpublish through the sharedmedia_file::store_hash_addressed_mp4helper into the never-evicted per-profilecanvas/dir), the Show Canvas toggle (useCanvasEnabled, default OFF), precedence (Canvas > motion cover > static cover),prefers-reduced-motion/ radio / Spotify exclusions, and theuseTrackCanvaslookup guards ({id, profileId}tag + per-track generation + monotonic profile-generation token) + frozen picker target.Docs-only, no code. The root README feature matrix was deliberately left unchanged — its UI row is already the widest cell, so adding Canvas would force a full-table re-pad for a summary line already covering the immersive now-playing experience.
Summary by CodeRabbit
Nouvelles fonctionnalités
Documentation