[Suggestion]: clarify behaviour of ref callbacks
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 72/100
- Type d'issue
- Documentation
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- javascript, react
- Domaine
- documentation
Piste de recherche
Commencez par la section ref-callback de la page react.dev liée et examinez le cas de test réduit de StackBlitz pour vérifier le comportement décrit. Clarifiez la phrase mise en évidence afin qu’elle reste exacte lorsque le callback renvoie une fonction de nettoyage, et assurez-vous que la formulation reste compréhensible sans dépendre de la note ultérieure.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
The described behaviour for ref callbacks with null as an argument is ambiguous.
Page
https://react.dev/reference/react-dom/components/common#ref-callback
Details
From https://react.dev/reference/react-dom/components/common#ref-callback:
React will also call your ref callback whenever you pass a different ref callback. In the above example, (node) => { ... } is a different function on every render. When your component re-renders, the previous function will be called with null as the argument, and the next function will be called with the DOM node.
If I understand correctly, the emphasised sentenced is not correct when the ref callback returns a clean up function, as is the case in the referenced example. This needs to be clarified.
There is a note later on which helps a bit, but I still think this sentence could be made clearer so it makes sense in isolation.
Reduced test case to confirm this behaviour: https://stackblitz.com/edit/vitejs-vite-2r9rn6m2?file=src%2FApp.tsx&terminal=dev
- Langage dominant
- JavaScript
- Étoiles
- 11.8k
- Forks
- 7.9k
- Merge moyen
- 16 h 6 min
- PR mergées (30 j)
- 7
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de reactjs/react.dev
-
type: documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
-
bug: unconfirmed
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
-
type: typos
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
-
bug: unconfirmed
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Toutes les issues de reactjs/react.dev
Issues similaires
-
ai-observability bug team/ai-observability
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
AllTheMods/ATM-10-L#19 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
SignalK/n2k-signalk#345 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
JuliaPluto/PlutoPlotly.jl#72 ·