redaxo / redaxo

REDAXO, a PHP-based CMS since 2004. Both simple and flexible.
https://redaxo.org
Other
330 stars 116 forks source link

Medienpool : image_extensions in der package.yml genauer definieren #1423

Open tbaddade opened 7 years ago

tbaddade commented 7 years ago

Ich muss zunehmend immer mehr Ausnahmen für SVGs notieren, damit diese nicht durch den MediaManager geschleust werden.

Daher müsste man erst einmal image_extensions genauer definieren. Sind hier Extensions enthalten die als <img /> ausgegeben werden können oder tatsächliche Rastergrafiken sein sollen, die auch durch den MediaManager gehen dürfen?

Vorschlag:

package.yml

Methoden

Ein Ausnahme wäre dann immer noch das gif. Hier müsste der MediaManager prüfen, ob das .gif Animationen enthält und entsprechend reagieren.

Übersicht der Formate https://de.wikipedia.org/wiki/Grafikformat

skerbis commented 7 years ago

EPS nicht vergessen.

tbaddade commented 7 years ago

EPS nicht vergessen.

Danke, hab es ergänzt.

alxndr-w commented 7 years ago

video_extensions: [mp4, mov, mpeg, wmv, gif] <- könnte man ja in dem Zuge auch Videoformate notieren und einen Player in den Medienpool einbauen. (ich lass .swf mal weg ;))

skerbis commented 7 years ago

@alexplusde m4v zähle ich auch dazu,wg. Player: https://github.com/redaxo/redaxo/issues/678

staabm commented 7 years ago

Ich muss zunehmend immer mehr Ausnahmen für SVGs notieren, damit diese nicht durch den MediaManager geschleust werden.

wo definierst du diese ausnahmen heute?

Daher müsste man erst einmal image_extensions genauer definieren. Sind hier Extensions enthalten die als ausgegeben werden können oder tatsächliche Rastergrafiken sein sollen, die auch durch den MediaManager gehen dürfen?

wo tritt dein problem heute auf? im backend oder im frontend ?

Anders gesagt: was ist überhaupt das problem was gelöst werden soll?

tbaddade commented 7 years ago

@staabm Ich habe eine eigene Wrapper Class für Medien und da prüfe ich öfters auf svg.

https://github.com/yakamara/yakme/blob/master/lib/Yakme/Media.php#L84-L87 https://github.com/yakamara/yakme/blob/master/lib/Yakme/Media.php#L121-L124

Hier musste ich jetzt wieder eine Ausnahme definieren, damit srcsets nich auf Vektorgrafiken angewendet wird. https://github.com/yakamara/yakme/blob/master/lib/Yakme/Media.php#L138-L142

Deshalb müsste man auch erst einmal genau definieren, was image_extensions tatsächlich heißen soll. Das Property wird ja auch im JS vom Medienpool genutzt.