Review Request 117277: KDatePicker: override changeEvent() instead of setEnabled()
Alex Merry
alex.merry at kde.org
Tue Apr 1 10:35:46 UTC 2014
-----------------------------------------------------------
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140401/ca16668e/attachment.html>
More information about the Kde-frameworks-devel
mailing list