Repository navigation
Align idle previews with desktop cadence and reduced motion - #25
Merged
Merged
Conversation
Retain the exact decoded artwork while pacing idle previews at 6.6 seconds. Provide static alternatives on the catalogue and every installer, with matching cache references, contributor guidance, and pinned preview validation in CI. Preserve authoritative atlases, package metadata, installer semantics, and immutable release assets.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (28)
📒 Files selected for processing (26)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change updates pet preview GIF timing to match the specified desktop-renderer durations and adds matching PNG stills. Website cards, installer pages, and pet READMEs now expose static previews. Validation checks preview assets and their references. ChangesPet preview assets and presentation
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The preview workflow has no identified merge-blocking issue; its validator runs through the documented script entry point. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 77.78% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 9 functions across 3 files. (23 skipped: 23 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
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
1 participant
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.
Summary
Idle GIFs played six times faster than the inspected desktop renderer, and reduced-motion CSS did not stop GIF playback. This change uses 6.6-second idle previews and static alternatives on the catalogue and all seven installers, preserving decoded artwork and authoritative package assets.
Scope: collection-wide preview/accessibility maintenance, documentation and validation. Files affected: package/site previews and PNGs, README cache references, catalogue/installers, preview tooling, requirements, CI and maintenance guidance.
Change type
Pet details
Package checklist
Validation performed
origin/main: seven pets, 75 local site references, 87 Markdown references.git diff --cached --check.Remote CI run 37103140374 passed catalogue validation, dependency installation, preview validation and whitespace checks for
8fca2e359bce5c314cff2d7c37f8d8a5f953628b. The PR still requires a qualifying approving review before normal merge; auto-merge is disabled. Pages publication/readback remain outstanding. No native overlay activation or pointer-behavior claim is made.Impact
Rights
Existing licensed artwork only; no new character artwork or third-party assets. Existing CC BY 4.0 attribution is preserved.
Summary by CodeRabbit