<div>hein created this revision.<br />
hein added a reviewer: graesslin.<br />
hein added a subscriber: plasma-devel.<br />
Restricted Application added a project: Plasma.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>Exposes closable state in the window model and adds tests. This was<br />
included in the protocol and interface, but missing from the model.</p>

<div style="margin: 16px 0;
            padding: 12px;
            border-left: 3px solid #2980b9;
            background: #daeaf3;"><span class="remarkup-note-word">NOTE:</span> I decided to call this one "Closeable" because that's what it
was already called in kwayland, however the e/no-e mess is getting
worse and there's no hope for fixing it because we have different
styles in API-stable code already. I'd prefer to do the right thing
for any added lines of code, which to me is naming it "Closable".</div>

<p>Martin, I think you need to make a maintainer's decision on this<br />
(doesn't matter which, as long as we pick one) to allow all of<br />
these revisions to progress.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Autotest extended.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWAYLAND KWayland</div></div></div><br /><div><strong>BRANCH</strong><div><div>moveresize</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1435" rel="noreferrer">https://phabricator.kde.org/D1435</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/client/test_plasma_window_model.cpp<br />
src/client/plasmawindowmodel.cpp<br />
src/client/plasmawindowmodel.h</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>hein, graesslin<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>