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

Contribuer Bug
1

How to reproduce:

  1. Go to one of your post and click Edit to open the composer. Don’t type anything.
  2. Press the Esc key => as expected, the composer disappears completely (it isn’t minimized, because the post hasn’t been modified).
  3. Refresh the browser page => the composer comes back.

In step 2, if you click the Cancel button instead of pressing the Esc key, the behavior is correct, i.e. the composer doesn’t come back after browser refresh.

5 « J'aime »
3

@jack2

I just did it, Yes it does exactly what you say. Not sure if it’s a bug or its suppose to do this actually.

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.