1136365 - Letter box video with black stripes

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect)

Description

[Nils Ohlmeier [:drno]]

Attachment: letterboxvideo.png

In my call today with mreavy her video suddenly got rendered in letter box format.

I actually like the 16:9 aspect ratio, better then the old format. But the black stripes at top and bottom are really ugly. Can we resize the conversation window to the size of the video, please?

Comment 1

[Mark Banner (:standard8)]

Nils, the screenshot is the popped-out window which you can resize yourself.

Have I misunderstood the issue here?

Comment 2

[Nils Ohlmeier [:drno]]

(In reply to Mark Banner (:standard8) from comment #1)
> Nils, the screenshot is the popped-out window which you can resize yourself.

Well I never resized it, besides clicking the arrow for popping it out. I'm not sure if the original window also had the black stripes. But Maire indicated that she even saw the black stripes in her own preview.

I'm not able to reproduce the problem if I call myself. Probably my build in cam does not provide letter box format.

I'll try to re-test with Maire.

Comment 3

[Nils Ohlmeier [:drno]]

Attachment: conversation169.png

Dang it I forgot to try to resize the popped out conversation window.

But as you can see from the attached screen shot, the "problem" of the black stripes, or in other words "loop video rendering areas not adjusting to aspect ratio of the incoming video stream", also exists with the small conversation window.

Comment 4

[Mark Banner (:standard8)]

Passing to Sevaan, though I don't think there's much we can do here - short of potentially dynamically resizing the window, which imo would feel a bit weird.

Comment 5

[Nils Ohlmeier [:drno]]

(In reply to Mark Banner (:standard8) from comment #4)
> Passing to Sevaan, though I don't think there's much we can do here - short
> of potentially dynamically resizing the window, which imo would feel a bit
> weird.

I'm actually in favor of dynamically resizing here. But I agree that resizing the small conversation window could feel strange.
What I think would be reasonable is to leave the small conversation window at its fixed initial size. But once the call is connected we should know the size and aspect ratio of the remote stream. So if someone pops out the conversation window after the call connected / someone joined the room (which always my use case), then it would be nice if we could make the aspect ratio of the popped out window depending on the aspect ratio of the remote video to avoid the "mourning stripes".

Comment 6

[Sevaan Franks [:sevaan]]

I like :drno's suggestion. Keep the conversation window as is, but if the window is popped out, resize it correctly.

Comment 7

[Mark Banner (:standard8)]

This is going to be fixed by the ux-refresh, or possibly one of the bugs surrounding that.

Comment 8

[Mark Banner (:standard8)]

(because we'll be using an object-fit: cover for the remote video)

Comment 9

[Mark Banner (:standard8)]

This is now fixed on desktop by bug 1180179.

Note: Local video may still have letterbox in the preview window - this is to ensure the user can see everything that may be seen by the receiver (privacy).