A visualização do Composer lado mostra os botões de controle abaixo dos vídeos enviados

Don · Agosto 11, 2024, 9:20pm

Olá

Estes botões não estão ativos, mas aparecem também abaixo dos vídeos.

Obrigado

awesomerobot · Agosto 13, 2024, 5:38pm

Não consigo reproduzir o problema, será que é um tipo de arquivo específico? Tentei MP4 e WebM sem sucesso

Don · Agosto 13, 2024, 6:03pm

Olá Aqui está um vídeo de exemplo. É um vídeo MP4. Ainda vejo os botões lá…

Ah, agora tentei carregar um vídeo mais longo e parece que ele só aparece por alguns segundos…

Don · Agosto 13, 2024, 6:19pm

Hmm, acabei de tentar em mais vídeos. Gravações de tela, etc. e parece que apenas aquele vídeo afeta agora do meu lado, estranho. Provavelmente é resultado de alguma conversão. Mas mesmo assim não é normal.

awesomerobot · Agosto 13, 2024, 6:42pm

ah ha, isso é meio bobo… mas é porque o nome do arquivo de vídeo contém dimensões, isso é capturado por uma função que serve para anexar metadados a uma imagem:

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

e esses metadados são usados para determinar se os controles devem aparecer

Don · Agosto 13, 2024, 6:50pm

De fato Boa observação, eu nunca teria percebido isso

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

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

awesomerobot · Agosto 13, 2024, 7:15pm

Tenho um PR pronto que corrigirá isso em breve:

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)

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

A visualização do Composer lado mostra os botões de controle abaixo dos vídeos enviados

Tópicos relacionados