> On June 7, 2012, 8:24 p.m., Jarosław Staniek wrote: > > kexi/plugins/forms/widgets/kexidbdatepicker.cpp, line 133 > > <http://git.reviewboard.kde.org/r/105179/diff/1/?file=66656#file66656line133> > > > > Don't we need just to call QWidget::hasFocus() when we need this info? > > Why we want to store and maintain this flag in extra place?
lineEdit is private in KDatePicker, so we can't jast call hasFocus() - Oleg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105179/#review14493 ----------------------------------------------------------- On June 7, 2012, 6:04 p.m., Oleg Kukharchuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105179/ > ----------------------------------------------------------- > > (Updated June 7, 2012, 6:04 p.m.) > > > Review request for Calligra, Jarosław Staniek and Adam Pigg. > > > Description > ------- > > Kexi Froms get new widget > > > Diffs > ----- > > kexi/plugins/forms/CMakeLists.txt 588fae903234da1e41aac1020e9ada01eeb06d08 > kexi/plugins/forms/kexidbfactory.cpp > 4d8412372d85c74347118da5f4fbda544f413b25 > kexi/plugins/forms/kexiformmanager.cpp > 521dfb62e303fb594daa33d0c58a7b17cc62cd5e > kexi/plugins/forms/widgets/kexidbdatepicker.h PRE-CREATION > kexi/plugins/forms/widgets/kexidbdatepicker.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/105179/diff/ > > > Testing > ------- > > tested on current master branch > > > Thanks, > > Oleg Kukharchuk > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
