FormidableLabs / FormidableLabs/nuka-carousel
Replace div elements for native button elements
- Dominant language
- TypeScript
- Stars
- 3.1k
- Forks
- 588
- PR merge metrics
- No merged PRs in 30d
Description
### Is there an existing issue for this?
- [X] I have searched the existing issues
### Code of Conduct
- [X] I agree to follow this project's Code of Conduct
### Feature Request
```markdown
- Benefits - focus, will be properly announce when using screen reader
- I would open a PR into your repo
```
https://github.com/FormidableLabs/nuka-carousel/blob/776b32f95fcfb75f3181bdf61f12afc5f377ffa6/packages/nuka/src/Carousel/NavButtons.tsx#L27
Contributor guide
Research direction
Start in packages/nuka/src/Carousel/NavButtons.tsx at the linked line and inspect how the current div elements handle navigation. Replace the relevant elements with native buttons while preserving their existing behavior. Done means the navigation controls retain their functionality and provide proper focus and screen-reader announcements.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- accessibility, frontend
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100