jack2
20 Diciembre, 2019 11:05
1
How to reproduce:
Go to one of your post and click Edit to open the composer. Don’t type anything.
Press the Esc key => as expected, the composer disappears completely (it isn’t minimized, because the post hasn’t been modified).
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 Me gusta
@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 Me gusta
j.jaffeux
29 Marzo, 2020 08:46
4
Ya no puedo reproducirlo. ¿Podrías verificar si sigue ocurriendo en try.discourse.org ?
jack2
29 Marzo, 2020 09:32
5
Sí, acabo de reproducirlo en try.discourse.org .
¡Espera! ¡Es aleatorio! A veces no ocurre.
j.jaffeux
29 Marzo, 2020 09:34
6
¿Puedes encontrar una reproducción consistente? Tengo una idea para una solución, pero preferiría contar con una reproducción consistente. Por favor.
1 me gusta
jack2
29 Marzo, 2020 09:38
7
Aquí tienes una reproducción consistente:
Ve a una de tus publicaciones y haz clic en Editar para abrir el editor. No escribas nada.
Espera unos segundos hasta que aparezca una marca de verificación en la esquina inferior derecha del editor
Presiona la tecla Esc => como era de esperar, el editor desaparece por completo (no se minimiza, porque la publicación no ha sido modificada).
Actualiza la página del navegador => el editor vuelve a aparecer.
2 Me gusta
j.jaffeux
29 Marzo, 2020 09:39
8
¡Genial! Vale, así que mi idea de solución probablemente funcione, muchas gracias, lo aprecio mucho
3 Me gusta
j.jaffeux
29 Marzo, 2020 12:39
9
Esto fue mucho más complicado de lo esperado. Creo que esto debería solucionarlo:
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.
He solicitado varias revisiones y podría tardar en fusionarse. Gracias de nuevo por el reporte detallado
5 Me gusta
j.jaffeux
30 Marzo, 2020 06:44
10
Actualización rápida: acabo de cerrar el PR. La solución funcionaba, pero vamos a intentar arreglarlo a un nivel superior.
Puede llevar un poco de tiempo
4 Me gusta
sam
31 Marzo, 2020 00:51
11
Hice esta corrección:
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.
Este archivo es súper complicado @eviltrout … no puedo esperar hasta que tengamos soporte para async, porque un gran montón de métodos a veces devuelven una promesa y a veces devuelven un objeto, lo que hace que sea muy difícil entender cómo funcionan las cosas.
Por ahora, introduje algunas promesas y rutas alternativas para que no guardemos un borrador inmediatamente después de hacer clic en el botón de editar. Esto también estaba volviendo loco a @supermathie , ya que hacía muy complicado y molesto copiar texto de N respuestas al OP.
7 Me gusta
Esto explica mucho de la frustración que estaba experimentando.
4 Me gusta
j.jaffeux
4 Abril, 2020 02:57
13
Este tema se cerró automáticamente 4 días después de la última respuesta. Ya no se permiten nuevas respuestas.
