Closed Bug 1532096 Opened 6 years ago Closed 5 years ago

[i3] picture in picture wrong _NET_WM_WINDOW_TYPE on Xorg/Linux

Categories

(Toolkit :: Video/Audio Controls, defect, P3)

67 Branch
defect

Tracking

()

RESOLVED DUPLICATE of bug 1553835

People

(Reporter: sdk, Unassigned)

References

Details

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:67.0) Gecko/20100101 Firefox/67.0

Steps to reproduce:

  1. Double-click right button mouse on a video
  2. Choose Picture in Picture

Actual results:

Picture in picture open as a tiled window in bspwm or i3 (they're both Tiling Window Manager on Linux) because _NET_WM_WINDOW_TYPE is set to NORMAL

Expected results:

_NET_WM_WINDOW_TYPE should be set to DIALOG so the Window Manager knows it needs to open it as a floating window.

Blocks: videopip

Ni Mike to prio this.

Flags: needinfo?(mconley)
Flags: needinfo?(mconley)
Priority: -- → P3
Blocks: 1532677
No longer blocks: videopip

I think changing PLAYER_FEATURES to include dialog (source) would fix this. It's quite likely however that changing the window type would cause other issues.

(In reply to Tom Schuster [:evilpie] from comment #2)

I think changing PLAYER_FEATURES to include dialog (source) would fix this. It's quite likely however that changing the window type would cause other issues.

I think you could also change the state of the window to make sure the Window Manager float the window by default https://specifications.freedesktop.org/wm-spec/1.3/ar01s05.html

Did bug 1543027 fix this?

Dialog should not be used on Linux - see Bug 1580733. We can craft the toplevel window to be on/top and floating by other way.

I guess _NET_WM_WINDOW_TYPE_DOCK or _NET_WM_WINDOW_TYPE_UTILITY is more suitable for the PIP window. Frankly I don't know how are those handled by i3.

Can you try to set a different _NET_WM_WINDOW_TYPE for the PIP window under i3 and see if that works for you? see
https://stackoverflow.com/questions/19035043/setting-x11-type-property-for-xmessage

Flags: needinfo?(contact)
Summary: picture in picture wrong _NET_WM_WINDOW_TYPE on Xorg/Linux → [i3] picture in picture wrong _NET_WM_WINDOW_TYPE on Xorg/Linux

Also PIP sets WM_NAME/WM_CLASS to Toolkit / PictureInPicture.

We can also set special WM_ROLE for that window to allow i3 to handle it specially. But this is more i3 window integrating bug than a general Firefox bug.

Also, can you please check how Chromium PIP window is handled on i3?

Thanks.

Can you try to set a different _NET_WM_WINDOW_TYPE for the PIP window under i3 and see if that works for you? see
https://stackoverflow.com/questions/19035043/setting-x11-type-property-for-xmessage

Doesn't seem to work. Also I use bspwm not i3. I also switched to Fedora on the laptop where I had that issue (It was running Debian 10). I don't know if something change recently in Nightly but it seems like on Fedora with the same bspwm version the PIP opens correctly as a floating window in the bottom right corner.

However, there still is one little problem. A small portion (~3px) of the right and bottom part of the PIP window is off-screen.

Flags: needinfo?(contact)

Can you try to set a different _NET_WM_WINDOW_TYPE for the PIP window under i3 and see if that works for you? see
https://stackoverflow.com/questions/19035043/setting-x11-type-property-for-xmessage

Doesn't seem to work.

It looks like i3 only looks at the _NET_WM_WINDOW_TYPE property once at time of window creation.
Changing the window type using xprop didn't work for me either.
However, changing it before window creation did result in a floating window.

This is implemented as a fix for bug 1553835.

A small portion (~3px) of the right and bottom part of the PIP window is off-screen.

I expect this is due to window borders (PIP window currently has titlebar/borders on i3).

Status: UNCONFIRMED → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE

(In reply to Dennis van der Schagt [:dennisschagt] from comment #9)

A small portion (~3px) of the right and bottom part of the PIP window is off-screen.

I expect this is due to window borders (PIP window currently has titlebar/borders on i3).

It doesn't have a border on bspwm where this issue happens. I've opened 1587869.

You need to log in before you can comment on or make changes to this bug.