Atalho de teclado "Esc" não funcionando corretamente no modo de edição

Contribuir Bug
1

Como reproduzir:

  1. Acesse um de seus posts e clique em Editar para abrir o editor. Não digite nada.
  2. Pressione a tecla Esc => conforme o esperado, o editor desaparece completamente (ele não é minimizado, pois o post não foi modificado).
  3. Atualize a página do navegador => o editor reaparece.

Na etapa 2, se você clicar no botão Cancelar em vez de pressionar a tecla Esc, o comportamento está correto, ou seja, o editor não reaparece após a atualização do navegador.

5 curtidas
3

@jack2

Acabei de fazer isso. Sim, faz exatamente o que você disse. Não tenho certeza se é um bug ou se deveria funcionar assim na verdade.

3 curtidas
4

Não consigo mais reproduzir isso. Você ainda consegue reproduzir isso em try.discourse.org, por favor?

5

Sim, acabei de reproduzir no try.discourse.org.

Espere! É aleatório! Às vezes não ocorre.

6

Você consegue encontrar uma reprodução consistente? Tenho uma ideia para uma correção, mas prefiro ter uma reprodução consistente. Por favor.

1 curtida
7

Aqui está uma reprodução consistente:

  1. Acesse uma de suas postagens e clique em Editar para abrir o editor. Não digite nada.
  2. Aguarde alguns segundos, até que um símbolo de verificação apareça no canto inferior direito do editor
    image
    image846×101 1.26 KB
  3. Pressione a tecla Esc => como esperado, o editor desaparece completamente (não é minimizado, pois a postagem não foi modificada).
  4. Atualize a página do navegador => o editor reaparece.
2 curtidas
8

Legal! Ok, então minha ideia de correção provavelmente vai funcionar, muito obrigado :heart:

3 curtidas
9

Isso foi muito mais complicado do que o esperado. Acho que isso deve resolver:

Solicitei várias revisões e pode demorar um pouco para ser mesclado. Obrigado novamente pelo relatório detalhado :+1:

5 curtidas
10

Atualização rápida: acabei de fechar o PR. A correção estava funcionando, mas vamos tentar resolver em um nível mais alto.

Pode levar algum tempo :slight_smile:

4 curtidas
11

Fiz essa correção:

Esse arquivo é super complicado, @eviltrout… Mal posso esperar até termos suporte para async, porque um grande conjunto de métodos às vezes retorna uma promessa e às vezes retorna um objeto, o que torna muito difícil raciocinar sobre as coisas.

Por enquanto, introduzi algumas promessas e alternativas para que não salvemos um rascunho imediatamente após clicar no botão de editar. Isso também estava deixando o @supermathie maluco, pois tornava muito complicado e chato copiar texto de N respostas para a OP.

7 curtidas
12

Isso explica muito da frustração que eu estava tendo. :clap:

4 curtidas
13

Este tópico foi automaticamente fechado 4 dias após a última resposta. Novas respostas não são mais permitidas.