popcodeorg / popcodeorg/popcode
Preview component should not rely on prop change detection to send expressions to frame
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 191
- Forks
- 143
- Merge moyen
- 4 j 11 h
- PR mergées (30 j)
- 5
Description
Currently the <Preview> component looks for changes in the list of console expressions to determine what if any expressions to send to the frame for evaluation. Component behavior that depends on both previous and current state is considerably harder to reason about and should be avoided. Instead, console entries should have state indicating whether they have been sent to a frame for evaluation; this should be updated synchronously after the frame initially posts the expression. The <Preview> component should simply consume all entries that are not marked as evaluation started.
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 suivre le composant et le console-entry state utilisé lorsque la frame poste une expression pour la première fois. Identifiez où les entrées sont marquées une fois l’évaluation commencée, puis mettez à jour le flux afin que Preview consomme les entrées qui ne sont pas marquées comme démarrées. C’est terminé lorsque la transmission de l’expression ne dépend plus de la comparaison des valeurs de props précédentes et actuelles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- frontend
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100