react-component / react-component/slider

Slider - handle prop should be optional

Offen
#653 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
TypeScript
Sterne
3.1k
Forks
768
Ø Merge
11 T. 22 Std.
Gemergte PRs (30 T.)
5

Beschreibung

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne bei src/Slider.tsx Zeile 29 und vergleiche die handle prop mit den Änderungen in pull request #641. Ermittle, ob die prop optional sein sollte, und überprüfe anschließend die resultierende Typisierung mit den relevanten Checks des Projekts.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
react, typescript
Bereich
frontend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.