react-component / react-component/slider
Slider - handle prop should be optional
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 3.1k
- Forks
- 768
- Avg merge
- 11d 22h
- Merged PRs (30d)
- 5
Description
If I read and understand the code correctly shouldn't the handle prop be optional? I might have also missunderstood. If so, please correct me.
https://github.com/react-component/slider/blob/b5a337e71c6678f2d24e018509379a8e6f1d0518/src/Slider.tsx#L29
Related to changes that came in this pull request https://github.com/react-component/slider/pull/641
Contributor guide
No contributing guide indexed for this repository
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 at src/Slider.tsx line 29 and compare the handle prop with the changes in pull request #641. Determine whether the prop should be optional, then verify the resulting typing with the project's relevant checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100