> On Dec. 10, 2013, 3:40 p.m., Sebastian Kügler wrote: > > So you mean that whether or not to set it depends on KWin's focuspolicy and > > should be taken into account in CompactApplet.qml?
no, it's just a very easy way to see when it should not be hidden. E.g. if you need to have the mouse inside the calendar than something is wrong. > On Dec. 10, 2013, 3:40 p.m., Sebastian Kügler wrote: > > src/declarativeimports/core/dialog.h, line 93 > > <http://git.reviewboard.kde.org/r/114378/diff/1/?file=223641#file223641line93> > > > > READ hideOnWindowDeactivate? > > > > Or is there a reason to use is*** here, such as an API collision? normal property naming scheme. if it is a bool it start's with "is". I don't mind if you want to have it different, but it's at least consistant with the other boolean properties. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114378/#review45460 ----------------------------------------------------------- On Dec. 10, 2013, 3:14 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/114378/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2013, 3:14 p.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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
