fix: sitewide accessibility — nav labels, table scope, aria-live, tablist, SVG#3249
fix: sitewide accessibility — nav labels, table scope, aria-live, tablist, SVG#3249
Conversation
…live, tablist labels, aria-hidden Bug 65: Add aria-label="Main navigation" to layout nav, aria-label="Breadcrumb" to directory/Breadcrumbs.tsx (wiki Breadcrumbs.tsx already had it). Bug 66: Add scope="col" to all <th> elements — SortHeader component covers sortable headers; standalone <th> elements in organizations-table get explicit scope. Add sr-only <caption> to organizations, ai-models, and people tables. Bug 67: Add role="status" aria-live="polite" to loading state <td> cells in organizations-table and people-table so screen readers announce loading changes. Bug 68: Add aria-label prop to ProfileTabs component (passed through to TabsList). Apply "Organization sections", "Person sections", and "Legislation sections" labels on respective detail pages. Default fallback is "Page sections". Bug 69: Add aria-label to "View all →" links on homepage — dynamic per section (e.g. "View all Organizations") and static "View all wiki pages" for recently updated. Bug 72: Add aria-hidden="true" to decorative SVG icons in layout (RSS feed icon), MobileNav (hamburger/close icons), and NavDropdown (chevron icon). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (13)
✨ 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 |
Test Coverage Report
Coverage is advisory only — no thresholds enforced. Generated by |
Summary
Fixes 6 P2 accessibility bugs from QA sweep (Discussion #3220):
aria-labelto all<nav>elements (Main navigation, Breadcrumb)scope="col"to all<th>viaSortHeader+ non-sortable headers; added<caption className="sr-only">to orgs/ai-models/people tablesrole="status"+aria-live="polite"to loading indicators in orgs/people tablesaria-labeltoProfileTabstablist ("Organization sections", "Person sections", "Legislation sections")aria-labelto "View all →" links on homepagearia-hidden="true"to decorative SVG icons in nav/mobile menu13 files changed, 35 insertions, 18 deletions.
Test plan
🤖 Generated with Claude Code