Review Request 117277: KDatePicker: override changeEvent() instead of setEnabled()

Christoph Feck christoph at maxiom.de
Tue Apr 1 19:49:27 UTC 2014


-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140401/3427fac2/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list