Composerプレビューサイドにアップロードされた動画の下にコントロールボタンが表示される

Don · 2024 年 8 月 11 日午後 9:20

こんにちは

これらのボタンはアクティブではありませんが、動画の下にも表示されます。

ありがとうございます

awesomerobot · 2024 年 8 月 13 日午後 5:38

問題を再現できません。特定のファイルタイプでしょうか？ MP4とWebMを試しましたが、うまくいきませんでした

Don · 2024 年 8 月 13 日午後 6:03

こんにちはサンプル動画はこちらです。MP4動画です。まだボタンが見えています…

ああ、今度は長い動画をアップロードしてみましたが、短い数秒の動画にしか表示されないようです…

Don · 2024 年 8 月 13 日午後 6:19

うーん、今度は動画でも試してみました。画面録画などです。そして、私の側ではその動画だけが影響しているようです。奇妙ですね。おそらく、何らかの変換の結果でしょう。しかし、それでも普通ではありません。

awesomerobot · 2024 年 8 月 13 日午後 6: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 日午後 6:50

確かに😯 よく見つけましたね、気づきませんでしたよ😅

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

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

awesomerobot · 2024 年 8 月 13 日午後 7:15

修正するPRが準備できました。\n\nUX: do not check for dimensions in video filename by awesomerobot · Pull Request #28349 · discourse/discourse · GitHub

トピック		返信	表示
Make it more obvious an image is resizable when uploading an image UX	10	1031	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 completed , insert-video	7	154	2023 年 10 月 26 日
Uploading of video file fails due to JS error Bug uploads , fixed	19	456	2024 年 8 月 27 日

Composerプレビューサイドにアップロードされた動画の下にコントロールボタンが表示される

関連トピック