Hallo zusammen,
Ich bin beim Aktualisieren des Mint-Themes aus dem offiziellen Git-Repository auf ein Problem gestoßen.
Umgebung
- Discourse-Version:
3.6.0.beta3-latest(tests-passed) - Theme-Quelle: GitHub - discourse/discourse-mint-theme: A modern theme with a hint of mint (Standard-Branch)
- Installiert über
/admin/customize/themes→ „Installieren“ → „Aus einem Git-Repository-URL“
Problem
Wenn ich im Admin-UI (/admin/customize/themes) auf „Auf neueste aktualisieren“ für das Mint-Theme klicke, schlägt das Update mit diesem Fehlerdialog fehl:
Die Theme-Screenshots müssen in einem der folgenden Formate vorliegen: .jpg, .jpeg, .gif, .png. Der Screenshot light.webp hat ein ungültiges Format.
In about.json verweist das Theme derzeit auf:
"screenshots": [
"screenshots/light.webp",
"screenshots/dark.webp"
]
Gemäß der Discourse-Theme-Dokumentation sind Theme-Screenshots jedoch nur als jpg, jpeg, gif oder png zulässig, weshalb .webp von der Core-Validierung abgelehnt wird.
Schritte zur Reproduktion
- Installieren Sie das Mint-Theme aus dem offiziellen Git-Repository unter /admin/customize/themes.
- Gehen Sie zum Eintrag des Mint-Themes.
- Klicken Sie auf „Auf neueste aktualisieren“.
- Beobachten Sie den Fehlerdialog, dass light.webp ein ungültiges Format hat.
Vorgeschlagene Lösung
Ich habe eine PR erstellt, die die Screenshots in PNG umwandelt und about.json entsprechend aktualisiert:
- screenshots/light.webp → screenshots/light.png
- screenshots/dark.webp → screenshots/dark.png
- about.json aktualisiert auf:
"screenshots": [
"screenshots/light.png",
"screenshots/dark.png"
]
Mit dieser Änderung wird das Theme im Admin-UI sauber aktualisiert und die Screenshots werden korrekt angezeigt.
Gerne passe ich die PR an, falls es eine bevorzugte Methode zur Handhabung von Screenshots gibt (oder falls .webp in Zukunft von Core unterstützt werden soll).