Scorciatoia da tastiera "Esc" non funziona correttamente in modalità modifica

Contribuisci Bug
1

Come riprodurre:

  1. Vai a uno dei tuoi post e clicca su Modifica per aprire l’editor. Non digitare nulla.
  2. Premi il tasto Esc => come previsto, l’editor scompare completamente (non è minimizzato, poiché il post non è stato modificato).
  3. Aggiorna la pagina del browser => l’editor ricompare.

Nello step 2, se invece di premere il tasto Esc clicchi sul pulsante Annulla, il comportamento è corretto, ovvero l’editor non ricompare dopo l’aggiornamento del browser.

5 Mi Piace
3

@jack2

L’ho appena fatto. Sì, fa esattamente quello che dici. Non sono sicuro se sia un bug o se sia previsto che accada effettivamente.

3 Mi Piace
4

Non riesco più a riprodurlo, riesci ancora a riprodurlo su try.discourse.org?

5

Sì, l’ho appena riprodotto su try.discourse.org.

Aspetta! È casuale! A volte non si verifica.

6

Riesci a trovare una riproduzione coerente? Ho un’idea per una correzione, ma preferirei avere una riproduzione coerente. Per favore.

1 Mi Piace
7

Ecco un caso riproducibile in modo coerente:

  1. Vai su uno dei tuoi post e clicca su Modifica per aprire l’editor. Non digitare nulla.
  2. Attendi qualche secondo, finché non appare un segno di spunta in basso a destra dell’editor
    image
    image846×101 1.26 KB
  3. Premi il tasto Esc => come previsto, l’editor scompare completamente (non viene minimizzato, poiché il post non è stato modificato).
  4. Aggiorna la pagina del browser => l’editor ricompare.
2 Mi Piace
8

Bene! Ok, quindi la mia idea di fix probabilmente funzionerà, grazie, molto apprezzato :heart:

3 Mi Piace
9

È stato molto più complicato del previsto, penso che questo dovrebbe risolvere il problema:

Ho richiesto diverse revisioni e potrebbe volerci del tempo per l’unione, grazie ancora per la riproduzione dettagliata :+1:

5 Mi Piace
10

Aggiornamento rapido: ho appena chiuso la PR. La correzione funzionava, ma proveremo a risolverla a un livello superiore.

Potrebbe richiedere un po’ di tempo :slight_smile:

4 Mi Piace
11

Ho applicato questa correzione:

Questo file è estremamente complicato, @eviltrout… Non vediamo l’ora che sia disponibile il supporto per async, dato che un gran numero di metodi restituisce a volte una promise e a volte un oggetto, rendendo molto difficile comprendere il comportamento del sistema.

Per ora ho introdotto alcune promise e bypassi in modo da non salvare una bozza immediatamente dopo aver cliccato il pulsante modifica. Anche questo stava mandando @supermathie fuori di testa, perché rendeva davvero complicato e fastidioso copiare il testo da N risposte nell’OP.

7 Mi Piace
12

Questo spiega molto della frustrazione che stavo provando, :clap:

4 Mi Piace
13

Questo argomento è stato chiuso automaticamente 4 giorni dopo l’ultima risposta. Non sono più consentite nuove risposte.