alexplusde / media_manager_responsive Goto Github PK

Es knallt mit der neusten Version:
/redaxo/src/addons/media_manager_responsive/lib/media_manager_type_meta.php:18
media_manager_type_meta::getMinWidth(): Return value must be of type int, string returned

Wenn ich auf :string caste knallt es danach woanders...

Guckst du drüber oder soll ich mich reinfuchsen?

PHP: 8.1.27
Redaxo: 5.17.1

Es wäre sehr nützlich wenn man den alt-Text beim Bildaufruf überschreiben könnte. Standardmässig wird ja der Bildtitel aus dem Medienpool geladen, manchmal ist es aber hilfreich, wenn man den benutzerdefiniert setzen könnte. Sowas in der Art:

rex_media_plus::get($entry['img'])->setAltText('Meine benutzerdefinierter Alt-Text')->getPicture('news-home');

oder so ähnlich

Es whoopst:

rex_media_plus::getAttributes(): Return value must be of type array, null returned.
So richtig raffe ich nicht warum?!

Aufruf:
echo rex_media_plus::get('entw_07_697x383.jpg')->getPicture('highlight-nav');

@iriswerner ich versuche das baldmöglichst umzusetzen für dich.

Die verknpüften Medientypen müssen in Bezug auf min-width von gross nach klein (zuerst die höchste min-width) sortiert werden, damit die korrekte grösse geladen wird (zumindest in Firefox und gemäss Picture bzw. Source Spezifikation, siehe https://html.spec.whatwg.org/multipage/embedded-content.html#attr-source-media und https://stackoverflow.com/a/41063084).

the browser goes over the list of sources and picks the first one that matches.

Funktioniert nicht

<picture>
  <source media="all AND (min-width: 1px)" sizes="" type="image/jpeg" data-width="400" data-height="105"
    srcset="/media/bild-md/bild.jpg?timestamp=1674658426">
  <source media="all AND (min-width: 480px)" sizes="" type="image/jpeg" data-width="800" data-height="210"
    srcset="/media/bild-xl/bild.jpg?timestamp=1674658426">
    <img type="image/jpeg"
    src="/media/bild-sm/bild.jpg?timestamp=1674658426" alt="" loading="lazy" width="400" height="105">
</picture>

Funktioniert

<picture>
  <source media="all AND (min-width: 480px)" sizes="" type="image/jpeg" data-width="800" data-height="210"
    srcset="/media/bild-xl/bild.jpg?timestamp=1674658426">
  <source media="all AND (min-width: 1px)" sizes="" type="image/jpeg" data-width="400" data-height="105"
    srcset="/media/bild-md/bild.jpg?timestamp=1674658426">
    <img type="image/jpeg"
    src="/media/bild-sm/bild.jpg?timestamp=1674658426" alt="" loading="lazy" width="400" height="105">
</picture>

Wahrscheinlich ist dir das klar, aber die Hilfeseite des Addons erklärt das nicht und bin erst selbst darüber gestolpert.

Native Lazyloading wäre ebenfalls ein cooles Feature :)

@ynamite weißt du, wie man das deklarieren muss, dass $media_plus auch eine Instanz von rex_media_plus ist?

Whooops nach update im Backend, danach nur noch im Safe-Mode aufrufbar.

Statt public function setClass(:string $class)
müsste es public function setClass($class) :string sein.
Nachdem ich das manuell angepasst habe, klappt's wieder :)

ParseError: syntax error, unexpected token ":", expecting variable
File: redaxo/src/addons/media_manager_responsive/lib/rex_media_plus.php
Line: 26

PR ist erstellt.

#22

In managed_media_responsive, um in der Installer-Suche weiter oben aufzutauchen.

Momentan wird nach Prio sortiert, berücksichtigt aber nicht die ratio, min/max-Werte. Das bedeutet bei nicht optimaler Konfiguration, dass das kleinste Bild und nicht das passende Bild geladen wird.

https://github.com/nucliweb/image-element#use-a-cdn-image-service

https://github.com/darylldoyle/svg-sanitizer

siehe redaxo/redaxo#5125 (comment)

https://github.com/redaxo/redaxo/blob/main/redaxo/src/addons/media_manager/lib/media_manager.php#L569

statt

In der aktuellen Version kommt es zu einem Fehler beim Aufruf der Funktion "rex_media_plus::get".

In der Version 1.0.1 habe ich ein Bild wie folgt in einem Modul eingebunden / ausgegeben:

$media = rex_media_plus::get("REX_MEDIA[1]"); 
$fe_img = $media->getPicture('bxslider'); 

Lt. Doku erfolgt die Einbindung wie folgt:
rex_media_plus::get("beispielbild.jpg")->getPicture($group);

Konkret heißt das:
$media = rex_media_plus::get("REX_MEDIA[1]")->getPicture('bxslider');

Im Frontend wird der folgende Fehler angezeigt:

TypeError thrown with message "implode(): Argument #2 ($array) must be of type ?array, string given"

PHP 8.0.27, Redaxo 5.14.2)

redaxo/redaxo#5292

Ansatz wie in https://github.com/FriendsOfREDAXO/media_manager_autorewrite/blob/master/boot.php wählen, aber nicht ein Bild durch den Medien Manager laufen lassen, sondern auf Wunsch in Picture-Element umwandeln.

echo rex_media_plus::get("beispielbild.jpg")->getBackgroundStyles($group); funktioniert nicht und gibt einen whooops.

In rex_media_plus.php habe ich die Methoe getBackgroundStyleswie folgt geändert:

public function getBackgroundStyles($group_name, $selector)
{
    return media_manager_type_group::getBackgroundStyles($this->getFilename(), $group_name, $selector);
}

und Aufruf dann wie folgt, dann klappts :)

echo rex_media_plus::get('REX_MEDIA[1]')->getBackgroundStyles('gruppenname', '#hintergrundbild');

Sehr cooles Addon btw.

Zum Debuggen, damit man weiß, aus welcher Gruppe die Profile zusammengestellt werden.

Danke @iriswerner für den Hinweis!