Accessibility review: comprehensive a11y improvements#82
Merged
brianorwhatever merged 2 commits intomainfrom Feb 8, 2026
Merged
Accessibility review: comprehensive a11y improvements#82brianorwhatever merged 2 commits intomainfrom
brianorwhatever merged 2 commits intomainfrom
Conversation
added 2 commits
February 8, 2026 08:06
- Add skip-to-main-content link for keyboard navigation (App.tsx) - Fix heading hierarchy: h2→h1 on Home and PriorityFocus pages - Add labels to form inputs: AddItemInput, SearchInput, RenameListDialog, ShareModal invite link - Add sr-only labels for screen readers on search and add-item inputs - Add role='search' to search container - Change search input type from 'text' to 'search' for semantics - Add role='dialog', aria-modal, aria-labelledby to RenameListDialog - Add aria-expanded to collapsible sections: CategoryHeader, ProvenanceSection, VerificationBadge - Add aria-label to toggle switches in Settings (dark mode, haptics, notifications) - Add aria-label to BatchOperations toolbar buttons (check, uncheck, delete, clear) - Add aria-label to ListCard links for screen reader context - Add aria-label to tag color picker buttons - Add aria-label to collaborator role selector - Add aria-label to ProvenanceInfo copy buttons - Add role='alert' and aria-live='assertive' to OfflineIndicator - Add id='main-content' to main element for skip link target
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
2 participants
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.
Accessibility Review (Phase 3)
Comprehensive accessibility audit and fixes across the app. All changes are additive (adding ARIA attributes, labels, roles) — no functional changes.
Changes
Keyboard Navigation
id="main-content"to main element as skip link targetHeading Hierarchy
Form Labels & Inputs
<label>(sr-only) +idto AddItemInput<label>(sr-only) +idandrole="search"to SearchInputtype="text"totype="search"for semanticshtmlFor/idpairing to ShareModal invite link inputDialog Accessibility
role="dialog",aria-modal,aria-labelledbyto RenameListDialogARIA States
aria-expandedto CategoryHeader collapse buttonaria-expandedto ProvenanceSection collapse buttonaria-expandedto VerificationBadge dropdownaria-labelto Settings toggle switches (dark mode, haptics, notifications)Screen Reader Support
aria-labelto BatchOperations toolbar buttonsaria-labelto ListCard links for contextaria-labelto tag color picker buttonsaria-labelto collaborator role selectoraria-labelto ProvenanceInfo copy buttonsrole="alert"+aria-live="assertive"to OfflineIndicatorTesting
bun run buildpasses ✅