telegram-react icon indicating copy to clipboard operation
telegram-react copied to clipboard

Videos don't fit on screen! Vertical axis

Open gkp1 opened this issue 4 years ago • 3 comments

I'm not sure if this is known, couldn't find in issues.

Videos recorded on phones (bad proportions) don't get resized to my correct screen size for some reason.

Resizing the whole webpage is needed to see the lower part of the video and video controls.

I have 1600x900 screen, on Windows, using firefox (latest version) only extension is ublock origin but it's turned off for the website. If more info is needed lmk!

attached images: normal resolution and 80% zoom to be able to see the whole thing

firefox_2020-10-15_17-54-48

2020-10-15_17-55-01 2020-10-15_17-55-13

gkp1 avatar Oct 15 '20 21:10 gkp1

Thanks for the report, will be fixed.

evgeny-nadymov avatar Oct 15 '20 21:10 evgeny-nadymov

@evgeny-nadymov Well, I did investigate this issue and I found a few workarounds:

  1. Changing the position attribute to absolute for .player-content. image

  2. Limiting the .player-content height to 80%, but I have no idea how well this could work with various videos in different aspect ratios. image

  3. Position attribute to absolute for .media-viewer-footer element. image

  4. Hiding the .media-viewer-footer-commands div which we don't want, I guess. image

  5. Position: absolute for .media-viewer-content.

Though, the captions are still chopped or are they intentionally missing? The best fix is the first since it keeps the to elements visible. Make note that I am no expert and quite new to CSS and all.

erayrafet avatar Jan 12 '21 15:01 erayrafet

@evgeny-nadymov Another apparent problem in this section of the app is that, in the instance where there are captions/text posted with an image or a video, they cover up the said file in the viewport. image Deploying a layout similar to Google Drive on wide-screen displays with the caption on the right would be a viable solution. Do let me know what you think. image

erayrafet avatar Mar 06 '21 08:03 erayrafet