fix(a11y): improve accessibility for Lighthouse score#2204
fix(a11y): improve accessibility for Lighthouse score#2204Ibochkarev wants to merge 2 commits intonuxt:mainfrom
Conversation
- Add @nuxt/a11y module for automated a11y checks - Add skip-to-main-content link and id on UMain - Add alt text to deploy page NuxtImg logos - Improve UInputCopy button aria-label (contextual) - Add browser tests for skip link presence and navigation
|
@Ibochkarev is attempting to deploy a commit to the Nuxt Team on Vercel. A member of the Team first needs to authorize it. |
…e formatting - Update aria-label for UInputCopy button to be more contextual - Refactor NuxtImg components in deploy pages for better readability and consistency
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis pull request enhances application accessibility by implementing WCAG compliance improvements across multiple components and the layout system. Changes include adding a skip-to-main-content navigation link in the default layout, introducing dynamic aria-labels based on component state in the copy input component, adding alt text attributes to image elements, installing the Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes 🚥 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)
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 |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Improve nuxt.com accessibility to raise Lighthouse a11y score from 83 toward 100 and address automatically detectable and manual a11y issues.
Motivation: Issue #1837 reports current Lighthouse a11y score of 83; the comment from @timdamen (Nov 2025) suggests using the nuxt/a11y module for auto-detection and fixing non-automatic issues.
Changes:
id="main-content"onUMainfor keyboard/screen-reader users.altto deploy index and deploy[slug]NuxtImglogos (deployment.title/provider.title+ “ logo”).aria-label="copy button"with a contextual label (e.g. “Copy command to clipboard” or “Copy<label>to clipboard”).href, target#main-content, and that clicking it scrolls main into view).Note: Browser tests run against
BASE_URL(default https://nuxt.com); the new skip-link tests will pass after this is deployed or when run with a localBASE_URL.Fixes #1837