feat(a11y): add route announcer (w/ limited noise on search)#1754
feat(a11y): add route announcer (w/ limited noise on search)#1754knowler merged 1 commit intonpmx-dev:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe change adds a NuxtRouteAnnouncer component to App.vue that handles route change announcements through a scoped slot. The component conditionally renders different text based on the active route: for the search route, it outputs a translated package title followed by " - npmx", whilst for all other routes it renders the standard route announcement message. This introduces accessibility-focused route change notifications via Nuxt's built-in announcer component. 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
fixes #1751