----------------------------------------------------------- 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 > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel