[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Review Request: Auto unhide panel when stuff in it gets activated
From:       "Marco Martin" <notmart () gmail ! com>
Date:       2009-01-24 10:51:08
Message-ID: 20090124105108.10710.56219 () localhost
[Download RAW message or body]



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

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]