В предпросмотре композитора под загруженными видео отображаются кнопки управления

Don · 11.Август.2024 21:20:02

Привет

Эти кнопки неактивны, но они также отображаются под видео.

Спасибо

awesomerobot · 13.Август.2024 17:38:02

Мне не удалось воспроизвести проблему. Возможно, это связано с конкретным типом файла? Я пробовал MP4 и WebM, но безрезультатно

Don · 13.Август.2024 18:03:06

Привет Вот пример видео. Это файл MP4. Я всё ещё вижу кнопки…

О, сейчас я попытался загрузить более длинное видео, и, похоже, отображается только короткое видео на несколько секунд…

Don · 13.Август.2024 18:19:01

Хм, только что попробовал ещё несколько видео, включая записи экрана, и похоже, что сейчас проблема касается только этого одного видео, что странно. Вероятно, это результат какой-то конвертации. Но даже в этом случае это не нормально.

awesomerobot · 13.Август.2024 18:42:36

Ага, это немного глупо… но дело в том, что в имени видеофайла указаны размеры, и это перехватывается функцией, предназначенной для добавления метаданных к изображению:

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

и эти метаданные используются для определения, должны ли отображаться элементы управления.

Don · 13.Август.2024 18:50:04

Действительно Хорошее замечание, я бы никогда этого не заметил(а)

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

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

awesomerobot · 13.Август.2024 19:15:14

У меня уже готов PR, который скоро это исправит:

github.com/discourse/discourse

UX: do not check for dimensions in video filename (#28349)

main ← ux-video-controls

merged 03:59PM - 15 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)

Тема		Ответов	Просм.
Make it more obvious an image is resizable when uploading an image UX	10	1067	07.10.2024
100% 75% 50% next to images in preview window UX	3	423	28.06.2024
Is there any way to check I chose the correct video without publishing the topic? Support	2	301	29.05.2024
Insert Video icon missing from composer UX insert-video , completed	7	170	26.10.2023
Uploading of video file fails due to JS error Bug uploads , fixed	19	596	27.08.2024

В предпросмотре композитора под загруженными видео отображаются кнопки управления

Связанные темы