[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: "Aaron Seigo" <aseigo () kde ! org>
Date: 2009-01-23 16:24:48
Message-ID: 20090123162448.24635.62295 () localhost
[Download RAW message or body]
-----------------------------------------------------------
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
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic