Review Request 114378: Add a hideOnWindowDeactivate property to PlasmaCore.Dialog

[Commit Hook]

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


This review has been submitted with commit 86d030eface9b88369d3903d15090b1b7388e0a7 by Martin Gräßlin to branch master.

- Commit Hook


On Dec. 11, 2013, 7:06 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114378/
> -----------------------------------------------------------
> 
> (Updated Dec. 11, 2013, 7:06 a.m.)
> 
> 
> Review request for Plasma and Sebastian Kügler.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This property is meant to bring back the functionality provided by PopupApplet. If the property is set to true the dialog gets hidden when it loses focus.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/dialog.h d612882 
>   src/declarativeimports/core/dialog.cpp 04ef7e4 
> 
> Diff: http://git.reviewboard.kde.org/r/114378/diff/
> 
> 
> Testing
> -------
> 
> Tested with CompactApplet in the desktop qmlpackages. It works, but it shouldn't be set to true there. It needs to be set only on the windows it should have. To test I recommend to switch to focus follows mouse. Whenever the window closes in an unexpected way, it's wrong.
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131211/093e3ed6/attachment.html>