<div>graesslin added a comment.</div><br /><div><div><p>I think all your ifdefs are not needeed. Neither are the x11 runtime checks.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div>imports/activitymanager/sortedactivitiesmodel.cpp:195-217 I don't understand why you made this conditional. It should also work on Wayland.</div><div>imports/activitymanager/sortedactivitiesmodel.h:25 Note: the code in kwindowsystem goes back to KDE 3 time and is clearly not written by me.</div><div>imports/activitymanager/sortedactivitiesmodel.h:36-43 NET Properties are not X11 specific. They are available on all platforms.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1009" rel="noreferrer">https://phabricator.kde.org/D1009</a></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>ivan, sebas, davidedmundson, mart, graesslin<br /><strong>Cc: </strong>broulik, plasma-devel<br /></div>