jack2
20 Dezembro , 2019 11:05
1
Como reproduzir:
Acesse um de seus posts e clique em Editar para abrir o editor. Não digite nada.
Pressione a tecla Esc => conforme o esperado, o editor desaparece completamente (ele não é minimizado, pois o post não foi modificado).
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 Curtiram
@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 Curtiram
j.jaffeux
29 Março , 2020 08:46
4
Não consigo mais reproduzir isso. Você ainda consegue reproduzir isso em try.discourse.org , por favor?
jack2
29 Março , 2020 09:32
5
Sim, acabei de reproduzir no try.discourse.org .
Espere! É aleatório! Às vezes não ocorre.
j.jaffeux
29 Março , 2020 09:34
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 Curtiu
jack2
29 Março , 2020 09:38
7
Aqui está uma reprodução consistente:
Acesse uma de suas postagens e clique em Editar para abrir o editor. Não digite nada.
Aguarde alguns segundos, até que um símbolo de verificação apareça no canto inferior direito do editor
Pressione a tecla Esc => como esperado, o editor desaparece completamente (não é minimizado, pois a postagem não foi modificada).
Atualize a página do navegador => o editor reaparece.
2 Curtiram
j.jaffeux
29 Março , 2020 09:39
8
Legal! Ok, então minha ideia de correção provavelmente vai funcionar, muito obrigado
3 Curtiram
j.jaffeux
29 Março , 2020 12:39
9
Isso foi muito mais complicado do que o esperado. Acho que isso deve resolver:
master ← jjaffeux:draft-escape-fix
closed 06:41AM - 30 Mar 20 UTC
This fix will also make sure the bootbox will be correctly shown when using esca… pe.
Test cases:
- edit post, wait for draft check, escape => should totally hide composer
- edit post, add a char, escape => should show bootbox, and reduce or hide depending on choice
Same behavior is expected when pressing on cancel.
Solicitei várias revisões e pode demorar um pouco para ser mesclado. Obrigado novamente pelo relatório detalhado
5 Curtiram
j.jaffeux
30 Março , 2020 06:44
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
4 Curtiram
sam
31 Março , 2020 00:51
11
Fiz essa correção:
committed 12:49AM - 31 Mar 20 UTC
When editing a post we were incorrectly saving a draft prior to user typing
Thi… s caused a bloat in the amount of drafts saved per user and inconsistency
around behavior of "escape" button.
It also lead to lots of warnings about draft conflicts when copying stuff
between posts.
The code is improved to use promises more appropriately, however further
changes are needed to clean up internals so methods consistently return
promises.
Too many methods in the controller sometimes return a promise and sometimes
an object. Long term the methods will become async and all of this will be
corrected.
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 Curtiram
Isso explica muito da frustração que eu estava tendo.
4 Curtiram
j.jaffeux
4 Abril , 2020 02:57
13
Este tópico foi automaticamente fechado 4 dias após a última resposta. Novas respostas não são mais permitidas.
