[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Martin Flöser]

https://bugs.kde.org/show_bug.cgi?id=389222

Martin Flöser  changed:

   What|Removed |Added

   Version Fixed In||5.15
  Latest Commit||https://commits.kde.org/kwi
   ||n/792d8404551d18827b08cf7af
   ||709dbb3cfbd4d53
 Status|REPORTED|RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Martin Flöser  ---
Git commit 792d8404551d18827b08cf7af709dbb3cfbd4d53 by Martin Flöser.
Committed on 31/12/2018 at 06:58.
Pushed by graesslin into branch 'master'.

Honor struts when placing Wayland transients

Summary:
So far transients were placed anywhere on the screen. This behavior was
inspired from X11 where context menus were able to overlap any other
window and use the complete screen area. On X11 context menus and
similar windows are override redirect and thus above all windows managed
by KWin.

On Wayland, though, context menus and similar and windows just like any
other window and thus follow stacking constraints like the parent
window. A context menu is stacked just above it's parent and is
(normally) below any panels. This resulted in problems that context menu
are stacked behind the panel with unreachable options.

This change changes the placement for transients to use the
PlacementArea instead of a screen geometry. Thus the transient does not
render behind the panel. Only in case of a fullscreen the struts are
ignored.
FIXED-IN: 5.15

Test Plan: New test case

Reviewers: #kwin

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D17826

M  +85   -1autotests/integration/transient_placement.cpp
M  +2-1placement.cpp

https://commits.kde.org/kwin/792d8404551d18827b08cf7af709dbb3cfbd4d53

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Martin Flöser]

https://bugs.kde.org/show_bug.cgi?id=389222

Martin Flöser  changed:

   What|Removed |Added

URL||https://phabricator.kde.org
   ||/D17826
  Flags||ReviewRequest+

--- Comment #6 from Martin Flöser  ---
Possible fix in https://phabricator.kde.org/D17826

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Patrick Silva]

https://bugs.kde.org/show_bug.cgi?id=389222

Patrick Silva  changed:

   What|Removed |Added

   Platform|Other   |Archlinux Packages
Version|5.12.90 |5.14.4

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Alexander Mentyu]

https://bugs.kde.org/show_bug.cgi?id=389222

Alexander Mentyu  changed:

   What|Removed |Added

 CC||notux...@gmail.com

--- Comment #5 from Alexander Mentyu  ---
Tested Dolphin on KDE neon live dev git unstable - bottom right click menu
almost always opens behind bottom panel without any QT_QPA_PLATFORM mode
modifications

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Roman Gilg]

https://bugs.kde.org/show_bug.cgi?id=389222

Roman Gilg  changed:

   What|Removed |Added

 CC||subd...@gmail.com

--- Comment #4 from Roman Gilg  ---
With >5.13 only reproducible when Dolphin is started in Wayland native mode
with:

QT_QPA_PLATFORM=wayland dolphin

(because in 5.13 the default changed to Xwayland mode for Qt apps, which don't
set it explicitly).

So context menus only wrong for Wayland native apps.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Dr . Chapatin]

https://bugs.kde.org/show_bug.cgi?id=389222

Dr. Chapatin  changed:

   What|Removed |Added

Version|5.12.4  |5.12.90

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Dr . Chapatin]

https://bugs.kde.org/show_bug.cgi?id=389222

Dr. Chapatin  changed:

   What|Removed |Added

Version|5.12.2  |5.12.4

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 389222] [Wayland] Context menu opens behind plasma panel

[Dr . Chapatin]

https://bugs.kde.org/show_bug.cgi?id=389222

Dr. Chapatin  changed:

   What|Removed |Added

Summary|[Wayland] Context menu  |[Wayland] Context menu
   |opens behing plasma panel   |opens behind plasma panel
Version|5.11.95 |5.12.2

-- 
You are receiving this mail because:
You are watching all bug changes.