Immagini in una chat ingrandita non possono essere aperte dopo aver inviato un messaggio con l'immagine

Passaggi per la riproduzione:

1. Ingrandisci la chat:

2.1. Invia un messaggio con un’immagine di medie o grandi dimensioni, in modo da aspettarti un popup con un’immagine più grande se ci clicchi sopra nel messaggio.

OPPURE

2.2. Ricevi un messaggio simile da altre persone nella chat.

3. Clicca sull’immagine. Non si aprirà.

Soluzione alternativa:

4. Aggiorna la pagina nel browser.

5. Clicca sull’immagine: ora si aprirà un popup con un’immagine di dimensioni maggiori.

2 Mi Piace

È una regressione recente, a quanto ne so?

1 Mi Piace

Sì, l’ho visto anche oggi. Sono abbastanza sicuro che sia un recente regressione.

1 Mi Piace

Penso che il problema sia qui:

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

Prima, api.decorateChatMessage passava l’elemento .chat-message-container:

Dopo, passa l’elemento .chat-cooked:

Con le immagini definite all’interno di chat-message-collapser-body, lightbox non riesce a trovare le immagini qui.

Modifiche introdotte qui: DEV: Refactor chat HTML decorating (#31309) · discourse/discourse@a0f681b · GitHub

4 Mi Piace

Sì, mi dispiace che tu non lo veda, è un sussurro ma ho già contattato @david al riguardo. Penso anche che sia correlato a questa recente modifica.

Grazie per aver controllato, non ho ancora avuto il tempo di dimostrare la mia teoria :+1:

4 Mi Piace

Grazie @meglio e @Arkshine!

Limitare il normale “decora cotto” solo al contenuto del messaggio è intenzionale. Ma abbiamo aggiunto una logica specifica di lightboxing che avrebbe dovuto gestire le immagini. Sembra che non abbia gestito il caso in cui l’immagine originale viene sostituita con la versione ottimizzata.

Correzione qui:

6 Mi Piace

Questo argomento è stato chiuso automaticamente dopo 21 ore. Non sono più consentite nuove risposte.