-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vertical video aspect ratio wrong in public view #1831
Comments
I confirm this. Same situation even with videos with landscape ratio video files shared by public link. Control buttons and part of the videos are not visible without scrolling down. Nextcloud 27.0.1, checked with Chrome and Firefox 115. |
Also confirmed. |
Same here on 28... |
Also on NC 29. |
In 27.1.11: If I do:
It seems to remedy the issue. Obviously I need to do some more testing but if it looks ok and the NC devs haven't already fixed it I'll create a pull request. |
I think I tracked it down to the Videos.vue file. It has the max-height attribute set to 100%. If I remove that, the video displays correctly across desktop and mobile. |
Could someone open a PR with a fix? :) |
Done: |
For those who don't want to wait the PR to be merged, you can fix the bug with the Custom CSS application and by putting this CSS: #preview video {
max-height: calc(100vh - var(--header-height) - 65px - 200px - 16px) !important;
} |
Steps to reproduce
Expected behavior
Video should be displayed in correct aspect ratio.
Actual behavior
Video gets displayed very large/zoomed in and you have to scroll to view the full video
Host OS
Ubuntu 22.04.2 LTS
Nextcloud AIO version
27.0.1
Current channel
Stable
Other valuable info
Only happend after upgrading from 27.0.0 to 27.0.1. In 27.0.0 everything works as expected.
Example file: https://cloud.philippjeschek.com/s/XgYj95qRCntm59d
The text was updated successfully, but these errors were encountered: