hein added inline comments.

INLINE COMMENTS

> davidedmundson wrote in taskgroupingproxymodel.cpp:724
> should be mostly a copy/paste right? Anything holding this up?

The sorting code will look at the first desktop in practice, which is why it 
should be the lowest desktop. I wasn't sure if I can rely on 
PlasmaWindow::plasmaVirtualDesktops() to return the desktops in 
position-sorting order. Something for @mart to answer.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D13745

To: hein, mart, mvourlakos, davidedmundson
Cc: anthonyfieroni, zzag, ngraham, abetts, plasma-devel, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart

Reply via email to