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