Review Request: Prevent KMessageBox instances with a parent from being application modal

[David Jarvie]

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


If the proposed patch is unacceptable, I suggest adding a new method

static void allowWindowModal(bool allow);

If called with true, this would enable the behaviour described in the apidocs for subsequent KMessageBox instances. By default, or if called with false, all KMessageBox instances would behave as now. As I said in an earlier comment, being able only to display application modal instances just isn't acceptable in some cases.

- David


On Aug. 7, 2011, 3:18 p.m., David Jarvie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102246/
> -----------------------------------------------------------
> 
> (Updated Aug. 7, 2011, 3:18 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> According to the apidocs, KMessageBox instances with a parent widget specified are supposed to be window modal, not application modal. This patch fixes this.
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kmessagebox.cpp 939be89 
> 
> Diff: http://git.reviewboard.kde.org/r/102246/diff
> 
> 
> Testing
> -------
> 
> Tested warningYesNo(), questionYesNoCancel() with a parent widget - widgets in a different window tree were still able to be used.
> 
> kmessageboxtest runs ok.
> 
> 
> Thanks,
> 
> David
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110808/5808417a/attachment.htm>