search

swinde / contao4-downloadarchive

1 stars 1 forks source link

Downloadbild: Responsive Images funktionieren nicht #17

Open wiphi opened 5 years ago

wiphi commented 5 years ago

Aktuell können zwar im Backend Modul für ein Downloadbild verschiedene Bildgrößen festgelegt werden, jedoch zeigen sie im Frontend keine Wirkung. Es wird immer nur eine Bildgröße geladen.

swinde commented 5 years ago

Hallo, ich habe versucht das Problem nachzustellen, kann aber erst mal nichts ungewöhnliches erkennen. Kannst Du das noch etwas näher erläutern!? Welche Contaoversion benutzt Du?

wiphi commented 5 years ago

Ich hab das ganze mal etwas aufbereitet und da jetzt auch die Website, in der das zum Einsatz kommt, offizell auf C4 läuft, kann ich auch das näher erläutern: Ich definiere im Backend für die Download-Images eine eigene Bildgröße im Theme mit verschiednen Pixeldichten:

2019-05-14 20_20_19-Start

Und weise diese einem Download-Item hinzu:

2019-05-14 20_19_07-Start

Dann sollte er im Frontend das picture_default.html5 Template laden und eine HTML Ausgabe in etwa so erstellen:

2019-05-14 20_31_48-Start

In unserem Fall generiert er aber nur dieses und lässt die wichtige srcset-Anweisung weg:

2019-05-14 20_22_18-Start

Problem hierbei ist das Template: https://github.com/swinde/contao4-downloadarchive/blob/4dadb499c070452ce192d7801b6512f69cfacfe9/src/Resources/contao/templates/ce_downloadarchive.html5#L14-L24 Da wird eine eigene Lösung verwendet und nicht das picture-Template geladen.

Ich würde generell die Templates trennen wollen, wie es auch in anderen Modulen gemacht wird: Ein Template für die Liste, ein weiteres für das einzelene (Download-) Element. Dies würde aber eine größere Änderung bedeuten.

swinde commented 5 years ago

Danke für deine Erklärung, jetzt verstehe ich was Du meinst. Allerdings kann ich das unmöglich lösen, ich stosse da an meine Grenzen! Hast Du da schon eine Lösung parat? Bei der Gelegenheit könnte man auch den Button für die 4te Bildposition (Bild unter dem Text) mit einbauen.

Ich würde generell die Templates trennen wollen, wie es auch in anderen Modulen gemacht wird: Ein Template für die Liste, ein weiteres für das einzelene (Download-) Element. Dies würde aber eine größere Änderung bedeuten.

Mache es nicht so kompliziert! Ich habe festgestellt, das Redakteure nicht verstehen, wie man ein Archiv aktualisiert, wenn eine neue Datei hochgeladen wurde.

wiphi commented 5 years ago

Deswegen habe ich mir auch erlaubt das gleich mir zuzuweisen ;) Die Änderung würde keine Auswirkungen für Redakteure haben. Sie betrifft nur Webentwickler bei Anpassung von Templates.

swinde commented 5 years ago

Deswegen habe ich mir auch erlaubt das gleich mir zuzuweisen ;)

Vielen Dank, bin gespannt!