<table><tr><td style="">hein updated this revision to Diff 42958.<br />hein edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-4dj23lreeohdzyx/">(Show Details)</a><br />hein added a comment.
</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/D14542">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Remove copied code.</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>Depends on D13887.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">The KCM currently installs alongside the old one with no<br />
file conflicts.<br />
<br />
</span>Remaining todo:<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
* To do the custom type demarshalling for the structs and vector<br />
  of structs I copied the type definitions and the functions for<br />
  now, but it would be good to put this stuff into a shared<br />
  header considering it's the same repo.</span><br />
* A small spacing conundrum with the RowLayout in the footer<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D14542?vs=42898&id=42958">https://phabricator.kde.org/D14542?vs=42898&id=42958</a></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D13887_2</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14542">https://phabricator.kde.org/D14542</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcmkwin/kwindesktop/CMakeLists.txt<br />
kcmkwin/kwindesktop/Messages.sh<br />
kcmkwin/kwindesktop/desktop.desktop<br />
kcmkwin/kwindesktop/desktopnameswidget.cpp<br />
kcmkwin/kwindesktop/desktopnameswidget.h<br />
kcmkwin/kwindesktop/desktopsmodel.cpp<br />
kcmkwin/kwindesktop/desktopsmodel.h<br />
kcmkwin/kwindesktop/kcm_kwin_virtualdesktops.desktop<br />
kcmkwin/kwindesktop/main.cpp<br />
kcmkwin/kwindesktop/main.h<br />
kcmkwin/kwindesktop/main.ui<br />
kcmkwin/kwindesktop/org.kde.kwin.virtualdesktopmanager.xml<br />
kcmkwin/kwindesktop/package/contents/ui/main.qml<br />
kcmkwin/kwindesktop/package/metadata.desktop<br />
kcmkwin/kwindesktop/virtualdesktops.cpp<br />
kcmkwin/kwindesktop/virtualdesktops.h</div></div></div><br /><div><strong>To: </strong>hein, mart, davidedmundson<br /><strong>Cc: </strong>davidedmundson, broulik, plasma-devel, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart<br /></div>