reactjs / reactjs/react.dev

Conflicting description on getSnapshotBeforeUpdate section on the React.Component page

Ouverte
#1,137 6 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
11.8k
Forks
7.9k
Merge moyen
1 j 11 h
PR mergées (30 j)
11

Description

Page: https://reactjs.org/docs/react-component.html
Section: getSnapshotBeforeUpdate

The description after the code block reads

In the above examples, it is important to read the scrollHeight property in getSnapshotBeforeUpdate because there may be delays between “render” phase lifecycles (like render) and “commit” phase lifecycles (like getSnapshotBeforeUpdate and componentDidUpdate).

It is indicating that we should be reading the scrollHeight property in getSnapshotBeforeUpdate due to the delays between the "render" phase lifecycles and "commit" phases lifecycles. But getSnapshotBeforeUpdate is indicated as one of the "commit" phases lifecycle. Is that correct?

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Lisez la section getSnapshotBeforeUpdate de la page React.Component à l’adresse https://reactjs.org/docs/react-component.html, y compris le bloc de code et l’explication entre guillemets qui se trouve en dessous. Vérifiez si la description distingue correctement les lifecycles des phases de render et de commit, puis mettez à jour la formulation contradictoire afin que le moment du lifecycle soit sans ambiguïté.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, react
Domaine
documentation
Type d'issue
Documentation
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.