<table><tr><td style="">graesslin added inline comments.
</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/D12820">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D12820#inline-67268">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mart</span> wrote in <span style="color: #4b4d51; font-weight: bold;">org_kde_plasma_virtual_desktop.xml:45</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">the virtualdesktop would then have an output property used for filtering?<br />
or, the layout event could be kept in the manager, but be not just rows and column, but pass the pair output and actual vector containing the virtualdesktop instances?</p>

<p style="padding: 0; margin: 8px;">also if it is per output then two desktops can be on the same row/column, as long as they are on different outputs..<br />
then also all the api for requesting/notifying activate/deactivate would need an output parameter.. /o\</p>

<p style="padding: 0; margin: 8px;">My primary use for this protocol was to make activities==virtual desktops (the quick way would be to just use the activity ids as desktop ids and use kactivitymanager to add/remove, so no need for api to ask kwin to add/remove desktops)<br />
this clashes a bit with different desktops per output, tough could be solved if i then make the activity id a separate property in the Virtualdesktop interface...</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Please note that your quick way would not work. Activities are disabled in KWin Wayland and I don't see them getting enabled again. Activities resulted in hard freezes several times and I don't trust them due to that. As maintainer I would need very convincing arguments on why activities won't break KWin again. Due to the fact that kamd speaks both dbus and windowing protocol I don't see a chance that KWin can rely on it.</p>

<p style="padding: 0; margin: 8px;">Due to the fact that activities used to block KWin it's an area which is completely not ported to Wayland yet. A Wayland window cannot be put on an activity. So this would require a significant amount of work. This also doesn't match anything we have in KWin to move forward. Activities have been in an unmaintained state since the start and the code is in a really bad state. None of the core KWin developers have touched it in the last year's. It's just a bad copy and paste of virtual desktops without any understanding of the domain. I do not want to see any future code being based on that. In KWin we started to evolve virtual desktops, so that in future it would be possible to implement the feature set of activities in virtual desktops. But for the time being it would mean X11 identifiers, that is numerical desktops.</p>

<p style="padding: 0; margin: 8px;">I put know out an interesting question: do we need layouts?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R127 KWayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12820">https://phabricator.kde.org/D12820</a></div></div><br /><div><strong>To: </strong>mart, KWin, Plasma, graesslin, hein<br /><strong>Cc: </strong>zzag, bshah, romangg, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>