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

Don · 2024年八月11日 21:20

您好

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

谢谢

awesomerobot · 2024年八月13日 17:38

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

Don · 2024年八月13日 18:03

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

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

Don · 2024年八月13日 18:19

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

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

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

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

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

awesomerobot · 2024年八月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	1124	2024年十月7日
100% 75% 50% next to images in preview window UX	2	459	2024年三月22日
Is there any way to check I chose the correct video without publishing the topic? Support	1	336	2024年五月29日
Insert Video icon missing from composer UX completed , insert-video	6	226	2020年七月24日
Uploading of video file fails due to JS error Bug uploads , fixed	18	722	2024年八月24日

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

相關主題