Review Request: Auto unhide panel when stuff in it gets activated

[Aaron Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/341/#review335
-----------------------------------------------------------



/trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelview.cpp
<http://reviewboard.vidsolbach.de/r/341/#comment277>

    i'd rather see this in setVisibilityMode so that this code isn't run when VisibilityMode is NormalPanel ... therwise, looks good.


- Aaron


On 2009-01-23 03:52:30, Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/341/
> -----------------------------------------------------------
> 
> (Updated 2009-01-23 03:52:30)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> This makes an hidden panel auto unhide when the panel containment emits activated(), the concept is that if an applet in it gets activated it probably has focus so would need keyboard input, is requesting user attention etc. for instance it would make easy for the taskbar to auto unhide the panel when a task wants attention.
> now i'm not sure whete this is located in the balance convenience/annoyance, but seemed a sensible idea...
> 
> it also need a patch to applet.cpp
> 
> Index: applet.cpp
> ===================================================================
> --- applet.cpp  (revision 915223)
> +++ applet.cpp  (working copy)
> @@ -1178,6 +1267,7 @@
>          d->activationAction->setObjectName(QString("activate widget %1").arg(id())); // NO I18N
>          connect(d->activationAction, SIGNAL(triggered()), this, SIGNAL(activate()));
>          connect(this, SIGNAL(activate()), this, SLOT(setFocus()));
> +        connect(this, SIGNAL(activate()), containment, SIGNAL(activate()));     
> 
>          QList<QWidget *> widgets = d->actions.associatedWidgets();
>          foreach (QWidget *w, widgets) {
> 
> that forwards activate of applets to activate of the containment
> yeah i know should checkout whole trunk/KDE so patches would be more decent, my bad :p
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelview.h
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelview.cpp
> 
> Diff: http://reviewboard.vidsolbach.de/r/341/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco
> 
>