-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/546/#review916
-----------------------------------------------------------


unfortunately, this breaks "passive" mode: if you click on a notification and 
then click off it, it goes away. ok, minor annoyance, right? open the clock ... 
the calendar pops up and then goes away as soon as you click back onto the 
document you were working on in the first place. we may need some event 
handling in the passive case to reject closes due to loss of focus.

- Aaron


On 2009-04-14 07:48:09, Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/546/
> -----------------------------------------------------------
> 
> (Updated 2009-04-14 07:48:09)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Use Qt::Tool window flag to prevent passive Plasma popups from showing over 
> screensaver.
> 
> 
> This addresses bug 179924.
>     https://bugs.kde.org/show_bug.cgi?id=179924
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/plasma/popupapplet.cpp 951469 
>   /trunk/KDE/kdelibs/plasma/private/popupapplet_p.h 951469 
> 
> Diff: http://reviewboard.kde.org/r/546/diff
> 
> 
> Testing
> -------
> 
> Tested passive popup with battery notifications.
> Checked that KDE menu correctly gets keyboard focus when opened.
> 
> 
> Thanks,
> 
> Aurélien
> 
>

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

Reply via email to