<div>graesslin created this revision.<br />
graesslin added a reviewer: Plasma.<br />
Restricted Application added a project: Plasma.<br />
Restricted Application added a subscriber: plasma-devel.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>Mostly meant for multi-screen setups: we don't want that a strut set on<br />
a window on screen 0 results in screen 1 completely being excluded. Even<br />
if that's strictly seen a client bug, it's better to just ignore the<br />
strut from KWin's side.</p>

<p>The sanity check is implemented in Client::adjustedClientArea.</p>

<p>From a pure standard point of view this change is a EWMH violation and<br />
thus can cause regressions: struts by clients no longer working.</p>

<p>A test case for struts is added, including some invalid combinations<br />
whose strut is ignored with this change.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>strut-sanity-checking</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1386" rel="noreferrer">https://phabricator.kde.org/D1386</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/wayland/CMakeLists.txt<br />
autotests/wayland/struts_test.cpp<br />
geometry.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>graesslin, Plasma<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>