La vista previa del compositor muestra los botones de control debajo de los videos subidos

Don · 11 Agosto, 2024 21:20

Hola

Estos botones no están activos pero también aparecen debajo de los videos.

Gracias

awesomerobot · 13 Agosto, 2024 17:38

No puedo reproducir el problema, ¿quizás es un tipo de archivo específico? He probado MP4 y WebM sin éxito

Don · 13 Agosto, 2024 18:03

Hola Aquí tienes un video de ejemplo. Es un video MP4. Todavía veo los botones ahí…

Oh, ahora intenté subir un video más largo y parece que solo aparece el corto de unos segundos…

Don · 13 Agosto, 2024 18:19

Hmm, ahora lo he probado en más vídeos. Grabaciones de pantalla, etc. y parece que solo ese vídeo me afecta ahora, extraño. Probablemente sea el resultado de alguna conversión. Pero aun así no es normal.

awesomerobot · 13 Agosto, 2024 18:42

ajaja, esto es un poco tonto… pero es porque el nombre del archivo de video contiene dimensiones, esto es captado por una función que está destinada a agregar metadatos a una imagen:

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";
            }

y estos metadatos se utilizan para determinar si deben aparecer los controles.

Don · 13 Agosto, 2024 18:50

¡Ciertamente Buena observación, nunca me habría dado cuenta

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

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

awesomerobot · 13 Agosto, 2024 19:15

Tengo una PR lista que solucionará esto pronto:

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)

Tema		Respuestas	Vistas
Make it more obvious an image is resizable when uploading an image UX	10	1031	7 Octubre 2024
100% 75% 50% next to images in preview window UX	3	387	28 Junio 2024
Is there any way to check I chose the correct video without publishing the topic? Support	2	284	29 Mayo 2024
Insert Video icon missing from composer UX completed , insert-video	7	154	26 Octubre 2023
Uploading of video file fails due to JS error Bug uploads , fixed	19	456	27 Agosto 2024

La vista previa del compositor muestra los botones de control debajo de los videos subidos

Temas relacionados