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

Martin Flöser <mgraess...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/b7ad4bcf881cdc184375a0c5b
                   |                            |64e2dd625e30344
             Status|UNCONFIRMED                 |RESOLVED
   Version Fixed In|                            |5.12.0

--- Comment #8 from Martin Flöser <mgraess...@kde.org> ---
Git commit b7ad4bcf881cdc184375a0c5b64e2dd625e30344 by Martin Flöser.
Committed on 10/01/2018 at 16:44.
Pushed by graesslin into branch 'master'.

Keep fullscreen windows in active layer based on transients not the group

Summary:
So far a not-active fullscreen X11 window was kept in the active layer if
the newly activated window is in the same group (that is same client
leader). For example a fullscreen X11 kwrite window is in the active layer
if another kwrite window is active. The two kwrite windows obviously
don't have anything to do with each other, but are in the same group.

This creates problems as it's not possible to raise other windows above
the active not-fullscreen kwrite window. E.g. the panel is stacked below.

The idea behind the check makes sense: if a fullscreen window opens
another window (e.g. a configuration dialog) it should not be put back
to normal layer. Thus the check is adjusted whether the new active
window is a transient to the fullscreen window. Thus the intention is
still the same, but does not cause the problems.

As the code now does not need to differentiate between X11 and Wayland
windows (group only on X11) the Client specific implementation is
removed and the method unvirtual'ed.
FIXED-IN: 5.12.0

Test Plan: Test passes

Reviewers: #kwin, #plasma

Subscribers: plasma-devel, kwin

Tags: #plasma

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

M  +1    -1    abstract_client.cpp
M  +1    -1    abstract_client.h
M  +4    -1    autotests/integration/x11_client_test.cpp
M  +0    -1    client.h
M  +0    -15   layers.cpp

https://commits.kde.org/kwin/b7ad4bcf881cdc184375a0c5b64e2dd625e30344

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

Reply via email to