L'anteprima del compositore mostra i pulsanti di controllo sotto i video caricati

Don · 11 Agosto 2024, 9:20pm

Ciao

Questi pulsanti non sono attivi ma appaiono anche sotto i video.

Grazie

awesomerobot · 13 Agosto 2024, 5:38pm

Non riesco a riprodurre il problema, si tratta forse di un tipo di file specifico? Ho provato MP4 e WebM senza successo

Don · 13 Agosto 2024, 6:03pm

Ciao Ecco un video di esempio. È un video MP4. Vedo ancora i pulsanti lì…

Oh, ora ho provato a caricare un video più lungo e sembra che appaia solo quello breve di pochi secondi…

Don · 13 Agosto 2024, 6:19pm

Hmm, ho appena provato su altri video. Registrazioni schermo ecc. e sembra che solo quel video influenzi ora dalla mia parte, strano. Probabilmente è il risultato di una conversione. Ma anche in quel caso non è normale.

awesomerobot · 13 Agosto 2024, 6:42pm

ah ah, è un po’ sciocco… ma è perché il nome del file video contiene le dimensioni, questo viene rilevato da una funzione che ha lo scopo di aggiungere metadati a un’immagine:

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 questi metadati vengono utilizzati per determinare se i controlli dovrebbero apparire

Don · 13 Agosto 2024, 6:50pm

Davvero Bella scoperta, non me ne sarei mai accorto

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

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

awesomerobot · 13 Agosto 2024, 7:15pm

Ho un PR pronto che risolverà questo problema a 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)

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

L'anteprima del compositore mostra i pulsanti di controllo sotto i video caricati

Argomenti correlati