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

Martin Gräßlin <mgraess...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|WONTFIX                     |FIXED
   Version Fixed In|                            |5.8.0
      Latest Commit|                            |http://commits.kde.org/plas
                   |                            |ma-workspace/f5d966ee77f43b
                   |                            |758f23eda02d879dd21442e5a2

--- Comment #69 from Martin Gräßlin <mgraess...@kde.org> ---
Git commit f5d966ee77f43b758f23eda02d879dd21442e5a2 by Martin Gräßlin.
Committed on 25/08/2016 at 11:44.
Pushed by graesslin into branch 'master'.

Allow struts on panels between screen edges if WM is KWin

Summary:
KWin starting with 5.7 supports struts on panels between screen edges.
Thus we can start setting struts on such panels, it won't exclude a
complete screen. But we don't know how other window managers handle it
and it's in general a rather "dangerous" change.

Thus to not affect other window managers, we check whether KWin is
running and only allow struts on thus panels if KWin is running.
Unfortunately we need to test this every time we go into the code path
as the WM might have changed.

In case the user replaces the window manager at runtime this still can
result in a bad situation.
FIXED-IN: 5.8.0

Test Plan:
Tested whether it works in general in X11. Further testing
needed by X11, multi-screen users.

Reviewers: #plasma

Subscribers: plasma-devel

Tags: #plasma

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

M  +68   -41   shell/panelview.cpp
M  +1    -0    shell/panelview.h

http://commits.kde.org/plasma-workspace/f5d966ee77f43b758f23eda02d879dd21442e5a2

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

Reply via email to