Don
11. August 2024 um 21:20
1
Hallo
Diese Schaltflächen sind nicht aktiv, erscheinen aber auch unter Videos.
Danke
3 „Gefällt mir“
Ich kann das Problem nicht reproduzieren, handelt es sich vielleicht um einen bestimmten Dateityp? Ich habe es mit MP4 und WebM versucht, aber ohne Erfolg
3 „Gefällt mir“
Don
13. August 2024 um 18:03
4
Hallo Hier ist ein Beispielvideo. Es ist ein MP4-Video. Ich sehe immer noch die Schaltflächen dort…
Oh, jetzt habe ich versucht, ein längeres Video hochzuladen, und es scheint, dass es nur für die wenigen Sekunden angezeigt wird…
2 „Gefällt mir“
Don
13. August 2024 um 18:19
5
Hmm, ich habe es jetzt mit weiteren Videos ausprobiert. Bildschirmaufnahmen usw. und es scheint, dass nur dieses Video jetzt auf meiner Seite betroffen ist, seltsam. Wahrscheinlich ist es das Ergebnis einer Konvertierung. Aber selbst dann ist es nicht normal.
2 „Gefällt mir“
ah ha, das ist irgendwie albern… aber es liegt daran, dass der Dateiname des Videos Abmessungen enthält, die von einer Funktion erfasst werden, die Metadaten zu einem Bild hinzufügen soll:
}
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";
}
und diese Metadaten werden verwendet, um zu bestimmen, ob die Steuerelemente angezeigt werden sollen.
3 „Gefällt mir“
Don
13. August 2024 um 18:50
7
Tatsächlich Gut bemerkt, das hätte ich nie erkannt


1 „Gefällt mir“
Ich habe einen PR vorbereitet, der dies bald beheben wird:
main ← ux-video-controls
opened 07:14PM - 13 Aug 24 UTC
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:

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:

6 „Gefällt mir“