fix(web): resolve 11 critical/high accessibility issues#2
Merged
Conversation
- Updated CLI versioning to use local version from package.json. - Improved URL options in CLI commands to clarify local app testing requirements. - Added smooth streaming buffer for provider output in App component. - Enhanced report loading to support nested report directories. - Refactored note loading to utilize a dedicated memory file. - Introduced debug logging for test sessions to capture provider output. - Implemented local app URL validation and error handling for test execution. - Updated OpenCode and ClaudeCode providers to include permission modes. - Refined environment variable handling for OpenCode provider.
- Add visible focus indicators (focus-visible outline) for all interactive elements - Add prefers-reduced-motion media query to disable animations - Add skip-to-content link in root layout - Fix all low-contrast text to meet WCAG AA 4.5:1 ratio - Add aria-label and aria-live region to copy button for screen readers - Add pause/resume control to terminal demo animation - Terminal demo respects prefers-reduced-motion (shows all lines statically) - Add aria-live region to terminal demo for screen reader announcements - Add aria-label to docs sidebar navigation - Create custom 404 page with navigation back to home and docs - Add id=main-content to main elements for skip link target
🤖 Augment PR SummarySummary: This PR improves accessibility across the GetWired marketing site and hardens the CLI’s local-testing workflow. Changes:
Technical Notes: The orchestrator now treats crawled site text as untrusted input and logs provider output to per-run debug logs for diagnosis. 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
… in report loading
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
Fixes 11 critical and high-severity accessibility issues across the GetWired marketing site.
Changes (7 files)
globals.cssprefers-reduced-motionmedia querylayout.tsxpage.tsxid="main-content", all contrast fixes (WCAG AA 4.5:1)install-command.tsxaria-label,aria-hiddenon icons,aria-livefor copy feedback, contrast fixterminal-demo.tsxaria-liveannouncements, contrast fixdocs/layout.tsxaria-labelon nav,id="main-content", contrast fixes, removed decorative pulsenot-found.tsxIssues Resolved
prefers-reduced-motionsupport (global CSS + terminal component)Testing
next buildpasses with no errorsPull Request opened by Augment Code with guidance from the PR author