Aperçu du compositeur : la barre latérale affiche les boutons de contrôle sous les vidéos téléchargées

Don · Août 11, 2024, 9:20

Bonjour

Ces boutons ne sont pas actifs mais apparaissent également sous les vidéos.

Merci

awesomerobot · Août 13, 2024, 5:38

Je n’arrive pas à reproduire le problème, s’agit-il d’un type de fichier spécifique ? J’ai essayé MP4 et WebM sans succès

Don · Août 13, 2024, 6:03

Bonjour Voici une vidéo d’exemple. C’est une vidéo MP4. Je vois toujours les boutons là…

Oh, j’ai maintenant essayé de télécharger une vidéo plus longue et il semble qu’elle n’apparaisse que pour la courte vidéo de quelques secondes…

Don · Août 13, 2024, 6:19

Hmm, je viens d’essayer sur d’autres vidéos. Enregistrements d’écran, etc. et il semble que seule cette vidéo affecte maintenant de mon côté, étrange. C’est probablement le résultat d’une conversion. Mais même dans ce cas, ce n’est pas normal.

awesomerobot · Août 13, 2024, 6:42

ah ha, c’est un peu idiot… mais c’est parce que le nom du fichier vidéo contient des dimensions, cela est capté par une fonction qui est censée ajouter des métadonnées à une image :

github.com/discourse/discourse

app/assets/javascripts/discourse-markdown-it/src/features/image-controls.js

9cadc402e


      
          }
          
          function isUpload(token) {
            return token.content.includes("upload://");
          }
          
          function hasMetadata(token) {
            return token.content.match(/(\d{1,4}x\d{1,4})/);
          }
          
          function appendMetaData(index, token) {
            const sizePart = token.content
              .split("|")
              .find((x) => x.match(/\d{1,4}x\d{1,4}(,\s*\d{1,3}%)?/));
            let selectedScale =
              sizePart && sizePart.split(",").pop().trim().replace("%", "");
          
            const overwriteScale = !SCALES.find((scale) => scale === selectedScale);
            if (overwriteScale) {
              selectedScale = "100";
            }

et ces métadonnées sont utilisées pour déterminer si les contrôles doivent apparaître

Don · Août 13, 2024, 6:50

En effet Bien vu, je n’aurais jamais remarqué ça

![SampleVideo1mb|video](upload://7cZrskeouBoDXn3EruQR8mFS3gN.mp4)

![SampleVideo_1280x720_1mb|video](upload://7cZrskeouBoDXn3EruQR8mFS3gN.mp4)

awesomerobot · Août 13, 2024, 7:15

J’ai une PR prête qui corrigera cela bientôt :

github.com/discourse/discourse

UX: do not check for dimensions in video filename

main ← ux-video-controls

opened 07:14PM - 13 Aug 24 UTC

awesomerobot

+10 -1

When adding a video/image/audio to a post, we check the token content for dimens…ions. If dimensions are found, we assume the content can be resized and give it image controls in the composer preview. The problem is that we check the entire token, including filename, for dimensions. This means if a video has dimensions in its filename, we show broken controls: ![image](https://github.com/user-attachments/assets/93f3bbc9-1c0f-4ac3-b541-aab854624824) This PR updates the `hasMetadata` function so it checks the area dedicated to dimensions specifically, which comes after the `|` in markdown. Now images get the controls, and videos with dimensions in their filename do not: ![image](https://github.com/user-attachments/assets/92d64c52-f4d5-4ecc-9567-c0737f724ae7)

Sujet		Réponses	Vues
Make it more obvious an image is resizable when uploading an image UX	10	1031	Octobre 7, 2024
100% 75% 50% next to images in preview window UX	3	387	Juin 28, 2024
Is there any way to check I chose the correct video without publishing the topic? Support	2	284	Mai 29, 2024
Insert Video icon missing from composer UX completed , insert-video	7	154	Octobre 26, 2023
Uploading of video file fails due to JS error Bug uploads , fixed	19	457	Août 27, 2024

Aperçu du compositeur : la barre latérale affiche les boutons de contrôle sous les vidéos téléchargées

Sujets connexes