<table><tr><td style="">hein updated this revision to Diff 42898.<br />hein edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-bkmhaov44tz3nj4/">(Show Details)</a><br />hein removed a subscriber: abetts.<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><p>Update description to scratch off the done todos</p></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>file conflicts.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">More importantly, here's the problems I found:<br />
* The `desktopCreated` signal doesn't have an index parameter,<br />
  which means I had to resort to appending new desktops I'm<br />
  told about through the protocol, even though `createDesktop`<br />
  takes a `position` parameter. The struct has an<br />
  x11DesktopNumber, but I don't want to use that.<br />
* The `position` parameter in `createDesktop` seems to be<br />
  handled incorrectly on the KWin side. I expected it to be<br />
  zero-indexed, but I have to add one to append a new desktop.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Remaining todo:</span><br />
* To do the custom type demarshalling for the structs and vector<div style="padding: 8px 0;">...</div>  header considering it's the same repo.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
* The KCM for some reason catches Return presses and closes<br />
  when anything in the Qt Quick view has focus. To do the<br />
  inline editing using a TextField in the delegate, I had to<br />
  implement `Keys.on(Return|Enter)Pressed` instead of using<br />
  `onAccepted`.</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=42897&id=42898">https://phabricator.kde.org/D14542?vs=42897&id=42898</a></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D13887_1</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/CMakeLists.txt<br />
kcmkwin/kwindesktopng/CMakeLists.txt<br />
kcmkwin/kwindesktopng/Messages.sh<br />
kcmkwin/kwindesktopng/desktopsmodel.cpp<br />
kcmkwin/kwindesktopng/desktopsmodel.h<br />
kcmkwin/kwindesktopng/kcm_kwin_virtualdesktops.desktop<br />
kcmkwin/kwindesktopng/org.kde.kwin.virtualdesktopmanager.xml<br />
kcmkwin/kwindesktopng/package/contents/ui/main.qml<br />
kcmkwin/kwindesktopng/package/metadata.desktop<br />
kcmkwin/kwindesktopng/virtualdesktops.cpp<br />
kcmkwin/kwindesktopng/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>