[A11y] Add Keyboard Navigation Support (Enter/Space) to Notification Preference Toggles
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 5
- Forks
- 7
- Avg merge
- 39m
- Merged PRs (30d)
- 39
Description
- Labels:
good first issue,accessibility,difficulty:beginner,area:core - Target File:
src/components/settings/notification-settings-card.tsx - Estimated Time: 10–15 minutes
- Difficulty: Level 1 (Beginner)
Why This Matters
Keyboard-only users must be able to toggle notification preferences using standard keyboard controls (Enter and Space).
Current Behavior
Interactive toggle containers in src/components/settings/notification-settings-card.tsx rely primarily on click events.
Requested Change
Add an onKeyDown event handler supporting event.key === 'Enter' || event.key === ' ' for keyboard toggle activation.
Acceptance Criteria
-
onKeyDownhandler implemented for toggle controls. -
tabIndex={0}present on interactive element containers. - Verification command passes:
npm run test:file -- tests/notifications.test.ts.
Verification
npm run test:file -- tests/notifications.test.ts
Contributor Notes
Accessibility and UX refinement.
🚀 How to Claim and Implement an Issue
- Choose an Issue: Browse our open issues and pick one matching your interest and available time.
- Comment on GitHub: Leave a comment on the corresponding GitHub issue: "I would like to work on this issue. Please assign it to me."
- Follow the Guide: Review our Beginner's Contribution Guide for step-by-step Git instructions.
- Verify Locally: Run the specific verification command listed on the issue before opening your PR.
- Submit PR: Open a pull request against
mainreferencing this issue number!
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in src/components/settings/notification-settings-card.tsx and inspect the interactive notification toggle containers. Add the requested keyboard handling and tabIndex behavior for Enter and Space, then run npm run test:file -- tests/notifications.test.ts. Done means the acceptance criteria pass and the notification toggles support keyboard activation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- accessibility, frontend
- Issue type
- Feature
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 94/100