Composer 预览侧边显示上传视频下方的控制按钮

Don · 2024 年8 月 11 日 21:20

您好

这些按钮没有激活，但也会出现在视频下方。

谢谢

awesomerobot · 2024 年8 月 13 日 17:38

我无法重现此问题，这是否是特定文件类型？我尝试了 MP4 和 WebM，但都没有成功

Don · 2024 年8 月 13 日 18:03

你好这是一个示例视频。这是一个 MP4 视频。我仍然能看到那些按钮……

哦，现在我尝试上传一个更长的视频，似乎它只显示了很短的几秒……

Don · 2024 年8 月 13 日 18:19

嗯，我刚又试了几个视频。录屏等。现在似乎只有那个视频对我这边有影响，很奇怪。可能是某种转换的结果。但即使那样，这也不正常。

awesomerobot · 2024 年8 月 13 日 18:42

啊哈，这有点傻……但这是因为视频文件名包含尺寸信息，这会被一个本应为图像附加元数据的函数捕获：

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 · 2024 年8 月 13 日 18:50

确实抓得好，我永远不会意识到这一点

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

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

awesomerobot · 2024 年8 月 13 日 19:15

我有一个准备好的 PR，很快就会修复这个问题：

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)

话题		回复	浏览量
Make it more obvious an image is resizable when uploading an image UX	10	1030	2024 年10 月 7 日
100% 75% 50% next to images in preview window UX	3	387	2024 年6 月 28 日
Is there any way to check I chose the correct video without publishing the topic? Support	2	284	2024 年5 月 29 日
Insert Video icon missing from composer UX insert-video , completed	7	154	2023 年10 月 26 日
Uploading of video file fails due to JS error Bug uploads , fixed	19	453	2024 年8 月 27 日

Composer 预览侧边显示上传视频下方的控制按钮

相关话题