Review Request 110504: Group tasks by activity

[José Millán Soto]

> On May 26, 2013, 9:42 a.m., Marco Martin wrote:
> > The libtaskmanager part is nice.
> > 
> > However the applet is about t be replaced, so the display part would have to be redone

I am aware that the tasks applet is going to be replaced soon, but most changes of this patch are in the library and not in the applet.
I don't mind creating a patch for the new version of the applet to handle this grouping once the new version of the applet is available.


- José


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110504/#review33145
-----------------------------------------------------------


On May 18, 2013, 2:13 p.m., José Millán Soto wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110504/
> -----------------------------------------------------------
> 
> (Updated May 18, 2013, 2:13 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> New grouping strategy was created to allow tasks to be grouped by activity.
> If an item is available in multiple activities, it will only appear in one of the activities the task is available on (except if it's available on all activities).
> GroupManager was modified so that whether items should be grouped just when the task bar is full or not is not only taken into account when grouping by program but also when grouping by activity.
> Activity icons are not handled yet, so an icon from the first task which was on the group is used as the icon of the task group.
> 
> 
> Diffs
> -----
> 
>   plasma/desktop/applets/tasks/tasks.cpp dbbb0cb 
>   libs/taskmanager/strategies/activitygroupingstrategy.cpp PRE-CREATION 
>   libs/taskmanager/groupmanager.cpp 9ac15e7 
>   libs/taskmanager/strategies/activitygroupingstrategy.h PRE-CREATION 
>   libs/taskmanager/CMakeLists.txt 70fa791 
>   libs/taskmanager/groupmanager.h ad4167a 
> 
> Diff: http://git.reviewboard.kde.org/r/110504/diff/
> 
> 
> Testing
> -------
> 
> Three activities were created (named "Activity 1", "Activity 2" and "Activity 3").
> One instance of KDialog was assigned to Activity 1, two instances of KDialog were assigned to Activity 2, three instances of KDialog were assigned to Activity 3 and one instance of KDialog was assigned both to Activities 2 & 3.
> The tasks applet was executed in plasma-windowed in all activities.
> Screenshot 1 & 2 show the task manager in the situation described above. Screenshot 3 shows the same task manager after leaving only one instance of KDialog in each activity ("Only group when taskbar is full" enabled), and screenshot 4 shows the task manager when the tasks are the same that in screenshot 3 but "Only group when taskbar is full" is disabled.
> 
> 
> File Attachments
> ----------------
> 
> Screenshot 1
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/18/img1.png
> Screenshot 2
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/18/img2.png
> Screenshot 3
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/18/img3.png
> Screenshot 4
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/18/img4.png
> 
> 
> Thanks,
> 
> José Millán Soto
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130526/ab5b3499/attachment.html>