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

Marco Martin notmart at gmail.com
Fri Jan 23 12:52:30 CET 2009


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

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



More information about the Plasma-devel mailing list