search

jweiland-net / video_shariff

This extension provides more privacy when embedding videos in frontend.
https://extensions.typo3.org/extension/video_shariff
Other
5 stars 6 forks source link

[feature request] Preview image size according to media adjustments #12

Closed Pjirlip closed 4 years ago

Pjirlip commented 5 years ago

Hallo,

erstmal vielen Dank für das kleine Plugin an das JWeiland-Team. Funktioniert ohne Problem, ich hätte aber noch einen Feature Request.

Wenn man für das Videoelement eine Breite und/oder Höhe einstellt nimmt der Preview-Container trotzdem die volle Breite seines Parent-Containers ein und erst nachdem man mit Klick zugestimmt hat, wird das Videoelement in der korrekt festgelegten Dimension dargestellt.

Es wäre meiner Meinung nach sinnvoll, wenn auch der Preview-Container die festgelegten Werte für Breite/Höhe des Media-Elements übernimmt, falls diese gesetzt sind.

Mit freundlichen Grüßen, PD

pascal20997 commented 5 years ago

Hallo Philipp,

das Vorschaubild sollte im Normalfall die Größeneinstellung übernehmen. Das ganze wird über den f:image ViewHelper geregelt: <f:image src="{previewImage}" width="{dimensions.width}" height="{dimensions.height}"/>

Lokal hatte ich mal ein Pfadproblem mit ddev. Die ImageMagick Komponenten liegen dort unter /usr/bin, während bei mir noch /usr/local/bin eingestellt waren und TYPO3 dann ohne Fehlermeldung einfach das Originalbild mit Originalgröße ausgegeben hat.

Kannst du mal im Quellcode schauen ob dein Vorschaubild unter typo3temp/assets/_processed_ liegt?

Pjirlip commented 4 years ago

Hallo Pascal,

vielen Dank für deine schnelle Rückmeldung, ImageMagick ist verfügbar und das Bild wird korrekt aus processed referenziert.

Bildschirmfoto 2019-10-28 um 15 41 28

Bei weiteren Versuchen ist mir aufgefallen, dass nur dimensions.width den Wert "falsch" setzt. Die Höhe wird korrekt ausgegeben. Weiter ist mir dann aufgefallen, dass er die Breite an die Höhe anpasst, scheinbar um das Verhältnis gleich zu behalten. Ich gehe stark davon aus, dass das Verhalten so gewollt ist. Dementsprechend kann das Issue geschlossen werden. Vielen Dank nochmal für deine Rückmeldung.