Reduce the height of portrait videos

This commit is contained in:
Michael 2024-03-05 21:25:00 +00:00
parent 3ad4ab2940
commit 31b92b16ed
2 changed files with 10 additions and 1 deletions

View file

@ -3643,6 +3643,13 @@ class Item
} }
if ($PostMedia->mimetype->type == 'video') { if ($PostMedia->mimetype->type == 'video') {
if (($PostMedia->height ?? 0) > ($PostMedia->width ?? 0)) {
$height = min(DI::config()->get('system', 'max_video_height'), $PostMedia->height);
$width = 'auto';
} else {
$height = 'auto';
$width = '100%';
}
/// @todo Move the template to /content as well /// @todo Move the template to /content as well
$media = Renderer::replaceMacros(Renderer::getMarkupTemplate('video_top.tpl'), [ $media = Renderer::replaceMacros(Renderer::getMarkupTemplate('video_top.tpl'), [
'$video' => [ '$video' => [
@ -3651,6 +3658,8 @@ class Item
'name' => $PostMedia->name ?: $PostMedia->url, 'name' => $PostMedia->name ?: $PostMedia->url,
'preview' => $preview_url, 'preview' => $preview_url,
'mime' => (string)$PostMedia->mimetype, 'mime' => (string)$PostMedia->mimetype,
'height' => $height,
'width' => $width,
], ],
]); ]);
if (($item['post-type'] ?? null) == Item::PT_VIDEO) { if (($item['post-type'] ?? null) == Item::PT_VIDEO) {

View file

@ -1,6 +1,6 @@
<div class="video-top-wrapper lframe" id="video-top-wrapper-{{$video.id}}"> <div class="video-top-wrapper lframe" id="video-top-wrapper-{{$video.id}}">
{{* set preloading to none to lessen the load on the server *}} {{* set preloading to none to lessen the load on the server *}}
<video src="{{$video.src}}" controls {{if $video.preview}}preload="none" poster="{{$video.preview}}" {else}preload="metadata" {{/if}}width="100%" height="auto"> <video src="{{$video.src}}" controls {{if $video.preview}}preload="none" poster="{{$video.preview}}" {else}preload="metadata" {{/if}}width="{{$video.width}}" height="{{$video.height}}">
<a href="{{$video.src}}">{{$video.name}}</a> <a href="{{$video.src}}">{{$video.name}}</a>
</video> </video>