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

[Marco Martin]

> On 2009-01-23 08:24:52, Aaron Seigo wrote:
> >
> 
> Diaa Sami wrote:
>     May be I'm missing something but there are some problems with this code, The major problem for me is that Containment doesn't have an activate() signal, so when I emit applet->activate() nothing happens ...
>     Other than that containment is a function, so the code needs two parenthesis to compile, this is not a problem, 2nd, this works only after a global shortcut is set(because this is where the code is executed), moving it to setVisibilityMode as Aaron suggested is probably the solution.
> 
> Diaa Sami wrote:
>     I'm new to reviewboard so I didn't notice the patch that Aaron sent, it made sense and looked like it should work, in theory at least.
>     Unfortunately after trying it doesn't, I call a method inside Applet that emit's activated() and nothing happens after that...

yeah, now i moved the activate() connection in Containment and had moved the connection in panlview to setVisibilityMode


- Marco


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


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
> 
>