Bug 333698 - newly mapped override redirect windows are above the active screen edges (affects eg. VLC fullscreen control panel)
Summary: newly mapped override redirect windows are above the active screen edges (aff...
Status: RESOLVED WORKSFORME
Alias: None
Product: kwin
Classification: Plasma
Component: platform-x11-standalone (show other bugs)
Version: 4.11.8
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-21 18:55 UTC by Kai Uwe Broulik
Modified: 2021-12-06 04:38 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:
mgraesslin: Wayland-
mgraesslin: X11+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kai Uwe Broulik 2014-04-21 18:55:14 UTC
When putting VLC player in fullscreen and expanding the controls bar (there is a little [←→] button next to the volume controls) so it spans the entire width at the bottom of the screen, the bottom screen edges stop working (the other ones, eg. at the top left corner are unaffected) so it seems the controls window is risen above even the screen edges.

Reproducible: Always

Steps to Reproduce:
1. Configure a hot screen corner bottom left or right
2. Open a video in VLC player
3. Go fullscreen
4. Click the [←→] expand button next to the volume controls
5. Try to activate one of your screen corners bottom left or bottom right
Actual Results:  
They don't work.

Expected Results:  
They are unaffected and still trigger.

xprop and clicking the controls window:
_NET_WM_SYNC_REQUEST_COUNTER(CARDINAL) = 92274818
_NET_WM_STATE(ATOM) = _NET_WM_STATE_ABOVE, _NET_WM_STATE_STAYS_ON_TOP
WM_TRANSIENT_FOR(WINDOW): window id # 0x5800002
_NET_WM_ICON(CARDINAL) =        [trafic cone icon]
_NET_WM_WINDOW_OPACITY(CARDINAL) = 3435973887
XdndAware(ATOM) = BITMAP
_MOTIF_DRAG_RECEIVER_INFO(_MOTIF_DRAG_RECEIVER_INFO) = 0x6c, 0x0, 0x5, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x10, 0x0, 0x0, 0x0
WM_CLIENT_LEADER(WINDOW): window id # 0x5800004
_NET_WM_PID(CARDINAL) = 12209
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_TOOLTIP, _KDE_NET_WM_WINDOW_TYPE_OVERRIDE, _NET_WM_WINDOW_TYPE_NORMAL
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x2, 0x1, 0x0, 0x0, 0x0
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW, WM_TAKE_FOCUS, _NET_WM_PING, _NET_WM_SYNC_REQUEST
WM_NAME(STRING) = "vlc"
WM_LOCALE_NAME(STRING) = "de_DE.UTF-8"
WM_CLASS(STRING) = "vlc", "Vlc"
WM_HINTS(WM_HINTS):
                Client accepts input or input focus: True
                Initial state is Normal State.
                bitmap id # to use for icon: 0x5800080
                window id # of group leader: 0x5800004
WM_NORMAL_HINTS(WM_SIZE_HINTS):
                user specified location: 0, 1014
                program specified location: 0, 1014
                user specified size: 1920 by 66
                program specified size: 1920 by 66
                program specified minimum size: 1920 by 66
                window gravity: Static
WM_CLIENT_MACHINE(STRING) = "kaiuwe-tragbar2"
WM_COMMAND(STRING) = { "vlc" }
_KDE_NET_WM_SHADOW(CARDINAL) = 92274755, 92274762, 92274769, 92274776, 92274783, 92274790, 92274797, 92274804, 60, 60, 60, 60
_KDE_NET_WM_USER_CREATION_TIME(CARDINAL) = 29115971
Comment 1 Thomas Lübking 2014-04-21 20:16:10 UTC
the panel is an override redirect, ie. outside managed stacking.
it appears on mouse moves on the fullscreen window and becomes upmost in the stack.

ScreenEdges woud have to be informed on unmanaged mappings and raise.

This would most likely immediately raise another bug report because SDL uses override redirects as well (and may not -or fail to- grab the mouse)

FYI:
- you can move the panel (just click and move it)
- don't ask me how i did that, but it does not span the full screen width here
Comment 2 Kai Uwe Broulik 2014-04-21 20:29:50 UTC
It only spans the entire width if you click the expand button next to the volume controls.

I got annoyed by moving the panel around by accident and stuff so I tried this but as the bug report indicates that broke my workflow. :-)
Comment 3 Thomas Lübking 2014-04-21 21:01:49 UTC
> It only spans the entire width if you click the expand button next to the volume controls.
What tells us when I stopped reading the bugreport and launched vlc for inspection =)
(i saw "xprop" and knew i wouldn't care in this regard ;-)

adding
     ScreenEdges::selft()->ensureOnTop();
in events.cpp ~:241 (UnmapNotify, at the end of the "if (e->xmap.override_redirect)" branch) should do, but will likely cause other trouble (esp. with plasma autohiding panels)
Comment 4 Martin Flöser 2014-04-22 05:37:16 UTC
> adding
>      ScreenEdges::selft()->ensureOnTop();
> in events.cpp ~:241 (UnmapNotify, at the end of the "if
> (e->xmap.override_redirect)" branch) should do, but will likely cause other
> trouble (esp. with plasma autohiding panels)

plasma autohiding panels won't be a problem any more in Plasma Next. I'm 
tempted to give a try to raise them on map/unmap notify of unmanaged in 
master.
Comment 5 Alexander Mentyu 2018-09-21 12:51:19 UTC
Can reproduce this bug on X11 - both with always visible and autohide panel
Cannot reproduce this bug on Wayland - also both with always visible and autohide panel

Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.14.80
Qt Version: 5.11.1
KDE Frameworks Version: 5.51.0
Kernel Version: 4.15.0-34-generic
Comment 6 kde.org 2021-11-06 21:49:22 UTC
This issue report is quite old. Can you please confirm, that it still persists with KDE 5.23?
Comment 7 Bug Janitor Service 2021-11-21 04:38:47 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 8 Bug Janitor Service 2021-12-06 04:38:11 UTC
This bug has been in NEEDSINFO status with no change for at least
30 days. The bug is now closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

Thank you for helping us make KDE software even better for everyone!