[Differential] [Request, 519 lines] D1386: Strut sanity checking: ignore struts which would exclude a complete screen

[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
  Mostly meant for multi-screen setups: we don't want that a strut set on
  a window on screen 0 results in screen 1 completely being excluded. Even
  if that's strictly seen a client bug, it's better to just ignore the
  strut from KWin's side.
  
  The sanity check is implemented in Client::adjustedClientArea.
  
  From a pure standard point of view this change is a EWMH violation and
  thus can cause regressions: struts by clients no longer working.
  
  A test case for struts is added, including some invalid combinations
  whose strut is ignored with this change.

REPOSITORY
  rKWIN KWin

BRANCH
  strut-sanity-checking

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

AFFECTED FILES
  autotests/wayland/CMakeLists.txt
  autotests/wayland/struts_test.cpp
  geometry.cpp

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

To: graesslin, Plasma
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160411/e0cd459b/attachment.html>