Las imágenes en un chat ampliado no se pueden abrir después de publicar un mensaje con la imagen

Pasos de reproducción:

1. Ampliar chat:

2.1. Publicar un mensaje con una imagen de tamaño mediano a grande, de modo que esperaría una ventana emergente con una imagen más grande si hiciera clic en ella en el mensaje.

O

2.2. Recibir un mensaje de este tipo de otras personas en el chat.

3. Hacer clic en la imagen. No se abrirá.

Solución alternativa:

4. Actualizar la página en el navegador.

5. Hacer clic en la imagen; ahora se abrirá una ventana emergente con una imagen de mayor tamaño.

2 Me gusta

¿Es esa una regresión reciente, que yo sepa?

1 me gusta

Sí, también lo he visto hoy. Estoy bastante seguro de que es una regresión reciente.

1 me gusta

Creo que el problema está aquí:

api.decorateChatMessage(
      (element) =>
        lightbox(element.querySelectorAll("img:not(.emoji, .avatar)")),
      {
        id: "lightbox",
      }
    );

Antes, api.decorateChatMessage pasaba el elemento .chat-message-container:

Después, pasa el elemento .chat-cooked:

Con las imágenes definidas dentro de chat-message-collapser-body, lightbox no puede encontrar las imágenes aquí.

Cambios introducidos aquí: DEV: Refactor chat HTML decorating (#31309) · discourse/discourse@a0f681b · GitHub

4 Me gusta

Sí, lamento que no lo veas, es un susurro pero ya le he avisado a @david al respecto. También creo que esto está relacionado con este cambio reciente.

Gracias por comprobarlo, aún no he tenido tiempo de demostrar mi teoría :+1:

4 Me gusta

¡Gracias @meglio y @Arkshine!

Limitar la decoración normal de “cocinado” solo al contenido del mensaje es intencional. Pero agregamos una lógica específica de “lightbox” que se suponía que manejaría las imágenes. Parece que eso no manejó el caso en el que la imagen original se reemplaza por la versión optimizada.

Corrección aquí:

6 Me gusta

Este tema se cerró automáticamente después de 21 horas. Ya no se permiten nuevas respuestas.