Review Request: Add a groupingstrategy to automatically group tasks that are grouped using kwin's windowtabbing

[Aaron J. Seigo]

> On Nov. 10, 2011, 11:30 a.m., Aaron J. Seigo wrote:
> > other than having to wait on the functionality in kdelibs (or performing the atom check directly in Task instead of relying on NetInfo), i wonder if this couldn't be merged into "Program Grouping" itself so that one does not need to choose between grouping by program or by tab collection. e.g. something along the lines of "group tabbed items together, everything else by program name" might fit the principle of least surprise and require people to do less configuration. thoughts?
> 
> Anton Kreuzkamp wrote:
>     It can't perform the atom check directly in in libtaskmanager as it also needs to get changes directly, which wouldn't be easy to do in ltm directly.
>     The merge with programgroupingstrategy would need some discussion, how it should exactly handle conflicts, and could irritate the user if not done correctly (not sure how to do it correctly), so I would prefer to let it wait till after the feature freeze (And then wait till 4.9 or push it as a bug-fix).

program grouping should be straightforward imho: if it is in a tabgroup, group it that way; for windows not in a tab group, group those by program name. the only possible edge case i can imagine is if one has 2, say, kwrite windows in one group and 3 in another: should they be grouped together? imho, no: the tab cllection should be followed strictly and only untabbed windows should be grouped by program name. simple :)

we have the time to make this right.


- Aaron J.


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


On Nov. 10, 2011, 3:59 p.m., Anton Kreuzkamp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103101/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2011, 3:59 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> This patch adds a groupingstrategy that groups tasks by their tab group, from kwin's window tabbing feature.
> Most of the code in wintabgroupingstrategy is copied from programgroupingstrategy.
> This patch is based on https://git.reviewboard.kde.org/r/103099/ to be included in KDELibs 4.7
> 
> 
> This addresses bug 224690.
>     http://bugs.kde.org/show_bug.cgi?id=224690
> 
> 
> Diffs
> -----
> 
>   libs/taskmanager/CMakeLists.txt c3848cf 
>   libs/taskmanager/groupmanager.h 760ddf1 
>   libs/taskmanager/groupmanager.cpp b66c6fa 
>   libs/taskmanager/strategies/wintabgroupingstrategy.h PRE-CREATION 
>   libs/taskmanager/strategies/wintabgroupingstrategy.cpp PRE-CREATION 
>   libs/taskmanager/task.h aa05176 
>   libs/taskmanager/task_x11.cpp 9b7d8cb 
> 
> Diff: http://git.reviewboard.kde.org/r/103101/diff/diff
> 
> 
> Testing
> -------
> 
> Tested without problems so far.
> 
> 
> Thanks,
> 
> Anton Kreuzkamp
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111110/2ead5d87/attachment-0001.html>