Les images dans une discussion agrandie ne peuvent pas être ouvertes après l'envoi d'un message contenant l'image

Bug
1

Étapes de reproduction :

1. Agrandir le chat :

image
image408×214 1.91 KB

2.1. Publier un message avec une image de taille moyenne à grande, de sorte que vous vous attendiez à une fenêtre contextuelle avec une image plus grande si vous cliquez dessus dans le message.

OU

2.2. Recevoir un tel message d’autres personnes dans le chat.

3. Cliquer sur l’image. Elle ne s’ouvrira pas.

Solution de contournement :

4. Rafraîchir la page dans le navigateur.

5. Cliquer sur l’image - elle ouvrirait maintenant une fenêtre contextuelle avec une image de plus grande taille.

2 « J'aime »
2

C’est une régression récente, à ma connaissance ?

1 « J'aime »
3

Oui, je l’ai aussi vu aujourd’hui. Je suis presque sûr que c’est une régression récente.

1 « J'aime »
5

Je pense que le problème se situe ici :

L'image montre un écran d'ordinateur affichant du code dans un éditeur de texte, avec une section d'exercice de codage mise en surbrillance où une déclaration de variable "this.getChatElement()" est en cours de saisie avec un caractère "I" en attente de complétion. (Légendé par l'IA)
L'image montre un écran d'ordinateur affichant du code dans un éditeur de texte, avec une section d'exercice de codage mise en surbrillance où une déclaration de variable "this.getChatElement()" est en cours de saisie avec un caractère "I" en attente de complétion. (Légendé par l'IA)1541×364 54.9 KB

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

Avant, api.decorateChatMessage transmettait l’élément .chat-message-container :

L'image affiche un fragment de code HTML structuré avec divers éléments div imbriqués, y compris un conteneur de message de chat avec des attributs de classe tels que "flex" et "grid". (Légendé par l'IA)
L'image affiche un fragment de code HTML structuré avec divers éléments div imbriqués, y compris un conteneur de message de chat avec des attributs de classe tels que "flex" et "grid". (Légendé par l'IA)1069×337 10.3 KB

Après, il transmet l’élément .chat-cooked :

Une interface utilisateur de message de chat avec une structure HTML visible. (Légendé par l'IA)
Une interface utilisateur de message de chat avec une structure HTML visible. (Légendé par l'IA)1134×330 20.7 KB

Avec les images définies à l’intérieur de chat-message-collapser-body, lightbox ne trouve pas les images ici.

Changements introduits ici : DEV: Refactor chat HTML decorating (#31309) · discourse/discourse@a0f681b · GitHub

4 « J'aime »
6

Oui, désolé que vous ne le voyiez pas, c’est un murmure mais j’ai déjà contacté @david à ce sujet. Je pense aussi que cela est lié à ce changement récent.

Merci d’avoir vérifié, je n’ai pas encore eu le temps de prouver ma théorie :+1:

4 « J'aime »
8

Merci @meglio et @Arkshine !

Limiter la décoration normale de « cuire » au seul contenu du message est intentionnel. Mais nous avons ajouté une logique spécifique de « lightbox » qui était censée gérer les images. Il semble que cela n’ait pas géré le cas où l’image d’origine est remplacée par la version optimisée.

Correction ici :

6 « J'aime »
9

Ce sujet a été automatiquement fermé après 21 heures. Les nouvelles réponses ne sont plus autorisées.