popcodeorg / popcodeorg/popcode

Preview component should not rely on prop change detection to send expressions to frame

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

Personne n'a encore pris cette issue.

code quality
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

  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

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

Recevez les nouvelles issues par e-mail

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