Le raccourci clavier "Esc" ne fonctionne pas correctement en mode édition

Contribuer Bug
1

Comment reproduire le problème :

  1. Accédez à l’un de vos messages et cliquez sur Modifier pour ouvrir l’éditeur. Ne tapez rien.
  2. Appuyez sur la touche Échap => comme prévu, l’éditeur disparaît complètement (il n’est pas minimisé, car le message n’a pas été modifié).
  3. Actualisez la page du navigateur => l’éditeur réapparaît.

À l’étape 2, si vous cliquez sur le bouton Annuler au lieu d’appuyer sur la touche Échap, le comportement est correct, c’est-à-dire que l’éditeur ne réapparaît pas après l’actualisation du navigateur.

5 « J'aime »
3

@jack2

Je viens de le faire. Oui, cela fait exactement ce que tu dis. Je ne sais pas si c’est un bug ou si c’est censé se passer ainsi en réalité.

3 « J'aime »
4

Je ne parviens plus à reproduire ce problème. Pouvez-vous toujours le reproduire sur try.discourse.org, s’il vous plaît ?

5

Oui, je viens de le reproduire sur try.discourse.org.

Attends ! C’est aléatoire ! Parfois, cela ne se produit pas.

6

Êtes-vous capable de trouver un scénario de reproduction constant ? J’ai une idée de correctif, mais je préférerais avoir un scénario de reproduction constant. S’il vous plaît.

1 « J'aime »
7

Voici une reproduction cohérente :

  1. Accédez à l’un de vos messages et cliquez sur Modifier pour ouvrir l’éditeur. Ne tapez rien.
  2. Attendez quelques secondes jusqu’à ce qu’une coche apparaisse en bas à droite de l’éditeur
    image
    image846×101 1.26 KB
  3. Appuyez sur la touche Échap => comme prévu, l’éditeur disparaît complètement (il n’est pas minimisé, car le message n’a pas été modifié).
  4. Actualisez la page du navigateur => l’éditeur réapparaît.
2 « J'aime »
8

Super ! Ok, alors mon idée de correction devrait probablement fonctionner, merci beaucoup, c’est très apprécié :heart:

3 « J'aime »
9

C’était beaucoup plus compliqué que prévu, je pense que cela devrait régler le problème :

J’ai demandé plusieurs avis et cela pourrait prendre du temps avant d’être fusionné, merci encore pour la reproduction détaillée :+1:

5 « J'aime »
10

Mise à jour rapide : je viens de fermer la PR. La correction fonctionnait, mais nous allons essayer de la résoudre à un niveau supérieur.

Cela pourrait prendre un peu de temps :slight_smile:

4 « J'aime »
11

J’ai appliqué cette correction :

Ce fichier est extrêmement délicat @eviltrout… J’ai hâte que nous ayons la prise en charge de async, car un grand nombre de méthodes renvoient tantôt une promesse, tantôt un objet, ce qui rend l’analyse du comportement très difficile.

Pour l’instant, j’ai introduit quelques promesses et contournements afin que nous ne sauvegardions pas un brouillon immédiatement après avoir cliqué sur le bouton Modifier. Cela rendait également @supermathie fou, car cela compliquait et rendait fastidieux le copiage de texte depuis N réponses vers le message original.

7 « J'aime »
12

Cela explique beaucoup de frustration que je ressentais, :clap:

4 « J'aime »
13

Ce sujet a été automatiquement fermé 4 jours après la dernière réponse. De nouvelles réponses ne sont plus autorisées.