Review Request: Start Present windows by ctrl+left click on a pager indicator

[Aaron Seigo]

> On 2010-08-28 14:56:56, Todd wrote:
> > Would it be possible to also do it with middle-click?  This is unused for that widget as far as I can tell, and would avoid needing to use the keyboard entirely.

that would be even more easy to trigger accidentally and raise the possibility of this being found not on purpose. imagine what someone who accidentally middle clicks and triggers this behaviour is going to think? most people won't connect "I just middle clicked on the representation of a window" with the present windows effect being triggered, and if they do it will likely come across as slightly odd. middle clicking doesn't have a "bring forward and show" semantic anywhere else afaik, ergo the unexpectedness.

so, no. :)

(tangentially, the same reason i don't like this feature is the same reason i veto'd the concept of scroll wheel over the battery plasmoid adjusting backlight brightness.)


- Aaron


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


On 2010-08-28 07:49:57, Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5035/
> -----------------------------------------------------------
> 
> (Updated 2010-08-28 07:49:57)
> 
> 
> Review request for Plasma, Aaron Seigo and Marco Martin.
> 
> 
> Summary
> -------
> 
> This patch adds support to pager to trigger the present windows effect for the desktop the user clicks on when holding ctrl. This is similar to what we have when clicking on a tasks group when ctrl is hold.
> 
> I'm not sure if this is a too hidden feature, but I consider it as could be useful and consistent with the tasks applet.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.cpp 1163977 
> 
> Diff: http://reviewboard.kde.org/r/5035/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100828/af615294/attachment-0001.htm