[Differential] [Request, 106 lines] D2164: Allow struts on panels between screen edges if WM is KWin

[graesslin (Martin Gräßlin)]

graesslin created this revision.
graesslin added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION 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.
  
  BUG: 71086
  FIXED-IN: 5.8.0

TEST PLAN
  Tested whether it works in general in X11. Further testing
  needed by X11, multi-screen users.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  struts-shared-panels

REVISION DETAIL
  https://phabricator.kde.org/D2164

AFFECTED FILES
  shell/panelview.cpp
  shell/panelview.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #plasma
Cc: plasma-devel, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160714/38a75065/attachment.html>