Clarify utility discovery and Sound setup - #105
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reachedNext included review available in 44 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (8)
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
The website's audio guide sent users to source builds without explaining how to open Sound in the module interface, and its general permission summary omitted Workspace Restore. Lead with the packaged Sound installation, explain the separate source-build flow, and distinguish those permissions.
Make the broader utility identity explicit in the hero, metadata, FAQ, about page and footers. Add direct links to all nine existing module sections and their source-build guide. Keep v1.0.0 Sound-only and all eight additional modules labeled in development. Fix guide overflow and text/focus contrast found during mobile review.
Change type
Risk areas
Static copy, anchor navigation and shared article styles. The public download remains v1.0.0. No app runtime, stored data, audio callback, native permission flow or dependencies change.
Verification
Real-time audio checklist
User-visible evidence
Before: the hero names only Sound, module discovery requires reading the full list, and the guide mixes packaged and source setup.
After: the hero names the broader utility scope, every module has a keyboard-accessible anchor, and packaged/source instructions are separate. Rendered desktop and mobile review covered all five pages at 1100px and 360px, plus homepage checks at 1440px and 390px. Nine keyboard link activations, skip navigation, reduced motion, release labels and FAQ structured-data consistency passed. Guide text and focus contrast are 5.46:1 and 16.28:1; long code stays within the mobile article.
Deployment
Production is rate limited. The existing quality-manager retry must wait until 2026-09-10 08:52 UTC unless Vercel explicitly clears the restriction. Merge to main is authorized. Production retries remain with the existing quality-manager monitor; no manual production build, redeploy or preview promotion is part of this change.
Attribution and license
Existing repository license and notice files are unchanged.