Review Request: taskmanager library: Manual Sorting Fix

Christian Mollekopf chrigi_1 at
Fri Sep 4 21:48:58 CEST 2009

This is an automatically generated e-mail. To reply, visit:

Review request for Plasma, Aaron Seigo and Marco Martin.


this fixes the manual sorting strategy, which is broken atm if the desktop is changed.

Since the manual sorting strategy relies on AbstractGroupableItem pointer not to change, we cannot remove it from the bookkeeping in case it returns (after a desktop change for instance).
I don't know if this is acceptable because this results in the item never being removed from the itemList until the groupmanager instance is deleted (lifetime of the applet which created the instance).

Another option would be to identify tasks and groups by WId, which is a bit more complicated, but if you think it would be better/cleaner, i could supply a patch.


  /trunk/KDE/kdebase/workspace/libs/taskmanager/groupmanager.cpp 1018615 



Tried it, works.



More information about the Plasma-devel mailing list