Review Request: Activate present windows from task group context menu

[Aaron Seigo]

> On 2009-06-27 04:53:14, Marco Martin wrote:
> > i like it!
> > i was also experimenting in the taskbar applet, (like with clicking on the group while pressing control) and mostly ended up with duplicate code, now, i am wondering:
> > what about providing a TaskGroupItem::presentWindows()? (that would basically be a carbon copy of PresentWindowsActionImpl) in this way it could be used by the context menu -and- by the applet..
> 
> Michal Dutkiewicz wrote:
>     Good idea. :-)
> 
> Ivan Cukic wrote:
>     Nice idea.
>     
>     As a side note, are we adding too many inconsistent and sometimes hidden features that have the same purpose?
>     
>     Group -> click -> menu of item in the group
>     Group -> hover -> popup with window thumbnails, click a thumbnail and it opens [obscure feature]
>     Group -> RMB -> menu item -> present windows
>     Group -> ctrl+click* -> present windows [hidden feature]
>     
>     Maybe we should have the /effect/ to be used when you click changeable by the user like it is for alt-tab, desktop switching...?
>

"Group -> hover -> popup with window thumbnails, click a thumbnail and it opens [obscure feature]"

we've actually received feedback from people that it "feels natural" to want to do that. and it makes sense: if you can see it, why can't you click on it? :)


- Aaron


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


On 2009-06-27 04:30:57, Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/897/
> -----------------------------------------------------------
> 
> (Updated 2009-06-27 04:30:57)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Patch adds a new menu entry to task group context menu to show the group of windows in kwin's present windows effect. When this entry is selected the window ids of the tasks listed in the group are passed to an XAtom which activates present windows effect for the passed windows (other windows will be hidden). The atom will be removed by present windows effect when it get's disabled.
> 
> This patch requires current kwin from trunk. What's missing is the detection if present windows effect is available or not (e.g. compositing not active, Compiz used instead of KWin). KWin announces support via setting atom on root window.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/libs/taskmanager/taskactions.cpp 972097 
>   trunk/KDE/kdebase/workspace/libs/taskmanager/taskactions_p.h 972097 
> 
> Diff: http://reviewboard.kde.org/r/897/diff
> 
> 
> Testing
> -------
> 
> Tested with both manual and automatic grouping.
> 
> 
> Thanks,
> 
> Martin
> 
>