search

FriendsOfREDAXO / focuspoint

Erweitert den Medienpool um die Fähigkeit, den Fokuspunkt eines Bildes zu bestimmen.
MIT License
58 stars 7 forks source link

Vorschau für Medienpool #30

Closed muenzberg-me closed 6 years ago

muenzberg-me commented 6 years ago

Sehr praktisch wäre es, wenn man im Medienpool beim setzen des Focuspoints direkt eine Vorschau für einen bestimmten Bildausschnitt bekommt.

Ich stell mir das etwa so vor:

  1. Auswahl eines Medientyps des Media Managers
  2. Einblenden der Vorschau auf Grunde der Daten des Medientyps.
olien commented 6 years ago

Find ich super!

eaCe commented 6 years ago

Eine Sache noch… Wenn ich das „echte“ Bild mit einem Focuspoint-Fit Effekt anzeigen live aktualisieren möchte, muss ich den Focuspoint vorher speichern. So würde ich bei jedem setzen des Focuspoints die Daten direkt speichern. Ich bin nicht sicher ob man das so machen sollte oder ob das einige verwirren könnte?

tbaddade commented 6 years ago

@eaCe muss ich den Focuspoint vorher speichern.

Nicht unbedingt. Du kannst die Werte im Effekt "focuspoint_fit" auch direkt anpassen (siehe Beispiel unten). Das Problem wird dann aber das Caching sein. Durch das Speichern des Formulars werden die bisherigen Cachedateien auf jedenfall gelöscht. Wie wäre es neben "Focuspoint zurücksetzen" einen Button "Focuspoint speichern" zu setzen. Der würde ebenfalls einfach das Formular abschicken.

Hier ist ein Beispiel wie ich die Höhen und Breiten einzelner Effekte anpasse.

Aufruf für den EP https://github.com/yakamara/yakme/blob/master/boot.php#L32

Dazugehöriger Code https://github.com/yakamara/yakme/blob/master/lib/Yakme/Effects/effect_responsive.php#L34-L54 Nicht wundern das die Methode in einem Effekt liegt. Ich wollte den Effekt und die Extension nur beisammen haben.

eaCe commented 6 years ago

Okay, habe es nun hinbekommen. Es funktioniert wie gedacht, allerdings muss ich den Speichervorgang noch optimieren... Danke @tbaddade ich nutze zum löschen des Bildcaches rex_media_manager::deleteCache

mediapreview4

eaCe commented 6 years ago

Nun ist aufgeräumt und die neue Test-Version liegt bei mir: https://github.com/eaCe/focuspoint/

Vielleicht kann @muenzberg-me diese ja noch ein mal testen. Wenn die Vorschau aktiv ist, wird der Focuspoint, wenn dieser verschoben wird, in der DB gespeichert. Bei einem Klick auf zurücksetzen wird dieser auch in der DB entfernt.

eaCe commented 6 years ago

Ich werfe nun einfach mal den PR rein, wenn jemand etwas auffällt wird man sich schon melden...

olien commented 6 years ago

Die Version ist aber noch nicht im Installer - oder?

(Gerade getestet. Gefällt mir sehr gut! Daumen hoch!!!)

eaCe commented 6 years ago

Nein noch nicht im Installer, ich dachte diesen schritt "muss" der Besitzer machen?

olien commented 6 years ago

Nö. Muß nicht :-) Wir sind alle die "Besitzer".

Ich würde vorschlagen, dass wir jetzt noch warten ob Chris sich zu meinem PR meldet und dann kommt das alles zusammen in den Installer. Oder?

LG Oliver

eaCe commented 6 years ago

Alles klar! :octocat: