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

[Aaron Seigo]

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


i really dislike such 'magic' features for the reason of trustability. a person will learn patterns of behaviour in the software and expect it to remain within those patterns. whenever the software deviates, the user learns to expect surprises and trust it less. this results in the user being more careful and paying more attention to what they click on, etc. rather than just forgetting about the environment and using it with as little conscious processing as possible. like an experience driver behind the wheel of a car. (which is why the idea of cars that accelerate on their own was so disruptive of an idea even outside of the idea of possible harm and injury.)

unfortunately, we apparently have this feature in the tasks widget already. :/ in fact, it seems to have been part of another commit, probably by accident:

   http://websvn.kde.org/?view=revision&revision=997990

that commit message says, "add a configuration ui for adding plasmoids into the systray: only ones marked X-Plasma-NotificationArea=true in their desktop file will show up in the list". it looks like the commit to taskgroup was an accident.

in fact, that it only applies to groups and not to individual windows, that it doesn't check for it being the root group, etc. makes me wonder about the general quality of the feature. those things could be improved in the tasks widget, of course, but personally i don't think this feature actually meets what we're trying to achieve in terms of "magic behaviours are bad".

it's been in there for a year, however, so i'm hesitant to yank it out at this point. and if we leave it in the tasks widget, we should make it consistent with the pager as well.

i'll leave it up to Marco since he was the one who made the original commit to the tasks widget.


trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.cpp
<http://reviewboard.kde.org/r/5035/#comment7364>

    view() isn't guaranteed; to be on the (probably overly) safe side, it should be:
    
    QGraphicsView *v = view();
    if (v) {
        Plasma::WindowEffect::...
    }
    
    same below


- Aaron


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/b16b21e3/attachment.htm