Review Request 117277: KDatePicker: override changeEvent() instead of setEnabled()
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/ --- Review request for KDE Frameworks and Christoph Feck. Repository: kwidgetsaddons Description --- KDatePicker: override changeEvent() instead of setEnabled() As the comment noted, setEnabled() is not virtual. Also, there was no need to separately enabled/disable the individual components, when they are children of the KDatePicker widget. Instead, we override changeEvent() and use it to focus the table (as before). This also extends the kdatepicktest to temporarily disable the picker. Diffs - src/kdatepicker.h 855aeb01ff457394cfbcff57845a57a9eae0a103 src/kdatepicker.cpp 345ee2d97632647789e6762e5e155e75802c2191 tests/kdatepicktest.cpp 5f29110f2335bb110b3470007a59f9cb8b6bcfee Diff: https://git.reviewboard.kde.org/r/117277/diff/ Testing --- Builds. Tests pass. Played around with kdatepicktest; behaviour seems unchanged. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117277: KDatePicker: override changeEvent() instead of setEnabled()
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/#review54813 --- Probably should only try to set the focus if the widget is going to be enabled? - Christoph Feck On April 1, 2014, 10:35 a.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/ --- (Updated April 1, 2014, 10:35 a.m.) Review request for KDE Frameworks and Christoph Feck. Repository: kwidgetsaddons Description --- KDatePicker: override changeEvent() instead of setEnabled() As the comment noted, setEnabled() is not virtual. Also, there was no need to separately enabled/disable the individual components, when they are children of the KDatePicker widget. Instead, we override changeEvent() and use it to focus the table (as before). This also extends the kdatepicktest to temporarily disable the picker. Diffs - src/kdatepicker.h 855aeb01ff457394cfbcff57845a57a9eae0a103 src/kdatepicker.cpp 345ee2d97632647789e6762e5e155e75802c2191 tests/kdatepicktest.cpp 5f29110f2335bb110b3470007a59f9cb8b6bcfee Diff: https://git.reviewboard.kde.org/r/117277/diff/ Testing --- Builds. Tests pass. Played around with kdatepicktest; behaviour seems unchanged. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117277: KDatePicker: override changeEvent() instead of setEnabled()
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/#review54816 --- Ship it! On the other hand, Qt hopefully won't remove the focus from a previous widget, if activating a disabled widget fails. - Christoph Feck On April 1, 2014, 10:35 a.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/ --- (Updated April 1, 2014, 10:35 a.m.) Review request for KDE Frameworks and Christoph Feck. Repository: kwidgetsaddons Description --- KDatePicker: override changeEvent() instead of setEnabled() As the comment noted, setEnabled() is not virtual. Also, there was no need to separately enabled/disable the individual components, when they are children of the KDatePicker widget. Instead, we override changeEvent() and use it to focus the table (as before). This also extends the kdatepicktest to temporarily disable the picker. Diffs - src/kdatepicker.h 855aeb01ff457394cfbcff57845a57a9eae0a103 src/kdatepicker.cpp 345ee2d97632647789e6762e5e155e75802c2191 tests/kdatepicktest.cpp 5f29110f2335bb110b3470007a59f9cb8b6bcfee Diff: https://git.reviewboard.kde.org/r/117277/diff/ Testing --- Builds. Tests pass. Played around with kdatepicktest; behaviour seems unchanged. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117277: KDatePicker: override changeEvent() instead of setEnabled()
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/#review54821 --- This review has been submitted with commit d1e542627f907c9512859c0f002210d9bf8b221b by Alex Merry to branch master. - Commit Hook On April 1, 2014, 10:35 a.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117277/ --- (Updated April 1, 2014, 10:35 a.m.) Review request for KDE Frameworks and Christoph Feck. Repository: kwidgetsaddons Description --- KDatePicker: override changeEvent() instead of setEnabled() As the comment noted, setEnabled() is not virtual. Also, there was no need to separately enabled/disable the individual components, when they are children of the KDatePicker widget. Instead, we override changeEvent() and use it to focus the table (as before). This also extends the kdatepicktest to temporarily disable the picker. Diffs - src/kdatepicker.h 855aeb01ff457394cfbcff57845a57a9eae0a103 src/kdatepicker.cpp 345ee2d97632647789e6762e5e155e75802c2191 tests/kdatepicktest.cpp 5f29110f2335bb110b3470007a59f9cb8b6bcfee Diff: https://git.reviewboard.kde.org/r/117277/diff/ Testing --- Builds. Tests pass. Played around with kdatepicktest; behaviour seems unchanged. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel