react-component / react-component/trigger
error of using goNextStatus in useVisibleStatus.ts
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 391
- Forks
- 245
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Checking goNextStatus() in useVisibleStatus.ts,
the source code look like this, for the switch condition, should we use prev instead of status?
function goNextStatus(callback?: () => void) {
cancelRaf();
rafRef.current = raf(() => {
// Only align should be manually trigger
setStatus((prev) => {
switch (status) { // should we use switch(prev) as it may be possible that react will combine multi setStatus together
case 'align':
return 'motion';
case 'motion':
return 'stable';
default:
}
return prev;
});
callback?.();
});
}
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
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.
Piste de recherche
Commencez par useVisibleStatus.ts et lisez goNextStatus, en particulier l’updater de setStatus et sa condition switch. Vérifiez comment React regroupe ces mises à jour d’état et si la valeur de la closure peut différer de la valeur prev de l’updater. Confirmez la progression prévue des états et validez le comportement avec les vérifications existantes pertinentes du dépôt, si elles sont disponibles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react, typescript
- Domaine
- frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100