Review Request: Prevent Plasma popups from showing over screensaver

[Aaron Seigo]

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