Rastrear un tema usando el atajo de teclado también rastrea otros temas

Cuando uso los atajos m t y m r para seguir/dejar de seguir un tema, esto también cambia el estado de seguimiento de otros temas que he visitado recientemente en la misma ventana.

Esto no parece ocurrir al usar el mouse.

Lo he observado usando Firefox 68.4.1esr. Lo noté por primera vez en los foros de Rust, y ahora también lo he verificado aquí.

5 Me gusta

¡Hola, gracias por el informe!

Creo que este error ha existido durante bastante tiempo; quizás los cambios recientes lo hayan hecho más visible.

En resumen:

  • no estamos validando el ID del tema al recibir el evento de cambio
  • no estamos limitando el atajo de teclado, lo que significa que si mantienes presionada la tecla “m”, se enviará el mismo evento indefinidamente…

Esto debería solucionarlo:

Esto debería ser fusionado antes de que termine la semana.

7 Me gusta

Hola,
¿Cuánto tiempo debería tardar desde la fusión de la solicitud de extracción hasta que se implemente la corrección?
En este momento, no observo ningún cambio en el comportamiento aquí.

Depende del foro. El commit se fusionó en master y se retrotraído a la rama estable, por lo que todos deberían poder obtenerlo.

4 Me gusta

Este tema se cerró automáticamente después de 3 días. Ya no se permiten nuevas respuestas.

@Falco, ¿esto ha regresado?

2 Me gusta

Hola @david
Sigo pudiendo reproducir el problema en este foro. No sé si entre tanto se haya vuelto irreproducible.

2 Me gusta

@joffreyjaffeux ¿podrías echarle otro vistazo, por favor?

@mpol no hemos podido reproducir esto. ¿Podrías explicarnos los pasos exactos que sigues para reproducirlo aquí en meta?

1 me gusta

Lo siento si resulta ser algo incorrecto de mi parte, pero aquí va, en Firefox 68.6.0esr justo ahora.

  1. Usando solo la navegación por teclado, comenzando desde una página principal recién abierta https://meta.discourse.org/ en una nueva pestaña, al inicio no tengo ningún tema seguido.
  2. Presiono ‘j’, ‘Enter’ para entrar en un tema.
  3. Presiono ‘j’ varias veces hasta los temas sugeridos, presiono ‘Enter’ para entrar en uno de ellos.
  4. Presiono ‘j’, ‘m’, ‘t’: ahora estoy siguiendo tanto este tema como el que ingresé desde la página principal.

Por cierto, acabo de notar que cuando en el paso 3 presioné ‘m’, ‘t’ inmediatamente, solo el tema correcto fue seguido. Pero incluso después de eso, presionar ‘j’, ‘m’, ‘t’ cambió el estado del tema anterior.

1 me gusta

Así que no pude reproducir este error exacto, y @david tampoco, pero @david tenía muchas solicitudes en su navegador, lo cual yo no tenía. La primera idea fue el navegador y luego ¿múltiples navegadores abiertos?

Después de buscar un poco, entendí que provenía del tamaño del navegador (que era estrecho para mí y amplio para David) y del botón de la línea de tiempo del tema, que seguía registrando nuevos manejadores de appEvents y se mostraba para David y no para mí.

Creo que esto debería mejorar considerablemente la situación en el seguimiento de temas; más detalles en el commit:

4 Me gusta

Ah, sí, las solicitudes. Entonces, al final, el procedimiento anterior generó dos POST idénticos a https://meta.discourse.org/t/146924/notifications y luego, supongo, el problemático, a https://meta.discourse.org/t/113192/notifications.

2 Me gusta

Sí, eso debería solucionar este tipo de cosas. Lo estoy fusionando ahora mismo para que puedas probarlo en Meta pronto.

5 Me gusta

@joffreyjaffeux Gracias. Tras un par de intentos, ya no pude reproducir el problema aquí y, de hecho, ahora siempre veo solo una solicitud correcta a /notifications, sin duplicados ni solicitudes para otros temas.

3 Me gusta