useEffect example could use something different like scrollY or scrollX.
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 11.8k
- Forks
- 7.9k
- Ø Merge
- 1 T. 11 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
The useEffect example in the "Synchronizing with Effects" could use a different example, maybe like a window.scrollX or a window.scrollY instead of e.clientX and e.clientY as they don't exist on a scroll event and cause it to show undefined on scroll event.
useEffect(() => {
function handleScroll(e) {
console.log(e.clientX, e.clientY);
}
window.addEventListener('scroll', handleScroll);
return () => window.removeEventListener('scroll', handleScroll);
}, []);
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Suchen Sie das useEffect-Beispiel auf der Dokumentationsseite „Synchronizing with Effects“ und prüfen Sie, wie dessen Scroll-Handler Event-Eigenschaften liest. Aktualisieren Sie das Beispiel so, dass es die im Issue beschriebene Scroll-Position verwendet, und überprüfen Sie anschließend, dass das gerenderte Beispiel für ein Scroll-Ereignis keine undefined-Werte mehr protokolliert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, react
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100