search

zweitag / middleman-images

Resize and optimize images for Middleman
MIT License
17 stars 3 forks source link

Feature image path #10

Closed pmk1c closed 7 years ago

pmk1c commented 7 years ago

Unterstützung von image_path. Da der helper nicht direkt in Middleman steckt, sondern aus padrino-helpers kommt, werden diese jetzt eingebunden.

resolves #5

pmk1c commented 7 years ago

Wir hatten das Problem, dass image_tag intern (im padrino-helper) image_path nutzt. Durch das Überschreiben von image_path wurde diese nun aufgerufen und wollte die Ressource, die schon bearbeitet war, ein weiteres mal bearbeiten.

Daher habe ich es nun so gelöst, dass im image_tag Helper nur die übergebenen Optionen gesichert werden, im image_path Helper wird dann das Entsprechende Processing durchgeführt.

Die andere Möglichkeit wäre einen neuen Ressourcentyp anzulegen (ProcessedImage). Auf diesen könnte man dann im image_path Helper testen. Habe ich auch mal durchgespielt. Dann haben wir allerdings ein Problem in dem Fall, dass image_tag aufgerufen wird, das Bild aber nicht bearbeitet (weder optimiert noch resized) werden soll.

Ist etwas vertrackt. Aber in dieser Form w läuft es auf jeden Fall.

pmk1c commented 7 years ago

Threads wären tatsächlich problematisch. Ich schau mir das nochmal an und such nach einer besseren Lösung.

pmk1c commented 7 years ago

siehe #12