<table><tr><td style="">hein created this revision.<br />hein added reviewers: Plasma, mart, davidedmundson.<br />hein added a subscriber: plasma-devel.<br />Restricted Application added a project: Plasma.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2379" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This introduces a new WindowTasksModel which is a QIdentityProxyModel<br />
around an instance of either WaylandTasksModel or XWindowsTasksModel.</p>

<p>This arbitration was previously done directly in TasksModel, along with<br />
refcounting to share a single window tasks sub-model between TasksModel<br />
instances. Factoring out both of these things has two goals:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">It allows users of libtaskmanager which are solely interested in window data (e.g. the Pager) to skip the unnecessary complexity of TasksModel, while still sharing the windowing monitoring with users of TasksModel.</li>
<li class="remarkup-list-item">TasksModel becomes fully free of windowing system-specific code and calls to KWindowSystem (to query the platform), making its code a little cleaner.</li>
</ul>

<p>The downside is another layer of indirection (the QIdentityProxyModel),<br />
but the upside is clear: The Pager wants to share window data and<br />
window monitoring with the Task Manager, but is not interested in<br />
startup notifications, does its own sorting (using the window stacking<br />
order), does not need grouping and has different filtering needs.</p>

<p>The Pager backend will use QSortFilterProxyModel around WindowTasksModel<br />
to mix in its own data and sort by stacking order.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2379" rel="noreferrer">https://phabricator.kde.org/D2379</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libtaskmanager/CMakeLists.txt<br />
libtaskmanager/tasksmodel.cpp<br />
libtaskmanager/windowtasksmodel.cpp<br />
libtaskmanager/windowtasksmodel.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, Plasma, mart, davidedmundson<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>