----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105628/#review16158 -----------------------------------------------------------
kwalletd/kwalletd.cpp <http://git.reviewboard.kde.org/r/105628/#comment12745> from API docs: "The usage of forceActiveWindow() is meant only for pagers and similar tools, which represent direct user actions related to window manipulation. Except for rare cases, this request will be always honored, and normal applications are forbidden to use it." I do not see that this is a pager or a taskbar. This means forceActiveWindow is the wrong way to do it. The way it is done here is an example of how to do a nag window. Please make it just a transient to the window it belongs to. KWin should take care of presenting it correctly. It is nothing which has to be above everything else. If we want that we should do it correctly, that is really block till the user entered the password. But in general I'm against windows behaving like that. It would make it very difficult for users to figure out what's going on and I think it's a very bad idea to ask for passwords in a nag window way as long as we have "kded asks for password". Yeah right, what's kded again? - Martin Gräßlin On July 20, 2012, 1:41 p.m., Allen Winter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105628/ > ----------------------------------------------------------- > > (Updated July 20, 2012, 1:41 p.m.) > > > Review request for KDE Runtime, David Faure and Fredrik Höglund. > > > Description > ------- > > This is an attempt to make the KWallet password prompt much harder to ignore > or miss. > > Now the prompt should always be in front of the parent window. and it should > unminimize if needed, and demand attention. > > > Diffs > ----- > > kwalletd/kwalletd.cpp 309c45f > > Diff: http://git.reviewboard.kde.org/r/105628/diff/ > > > Testing > ------- > > Just using it in various scenarios. > For example, if the akonadi maildispatcher needs to open kwallet now the > password prompt is always in front of kmail > > > Thanks, > > Allen Winter > >
