[Kde-pim] Review Request: Signal changes in KDateEdit and KTimeEdit (the first of two related review requests)

Bertjan Broeksema broeksema at kde.org
Fri Sep 24 09:17:24 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5432/#review7738
-----------------------------------------------------------

Ship it!


Looks good to me.

- Bertjan


On 2010-09-23 21:09:27, Anselmo Melo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5432/
> -----------------------------------------------------------
> 
> (Updated 2010-09-23 21:09:27)
> 
> 
> Review request for KDE PIM and David Faure.
> 
> 
> Summary
> -------
> 
> This is the first of two review requests about changing widgets in incidenceeditor-ng for korganizer-mobile.
> 
> In this review request, we introduce changes in KTimeEdit and KDateEdit, both from kdepim/libkdepim.
> 
> As part of our current task in korganizer-mobile, we want to use KTimeEdit and KDateEdit instead of QTimeEdit and QDateEdit in incidenceeditor-ng.
> 
> Why?
> With the new Time and Date selector widgets designed by Nuno, we do not need the spin buttons anymore (QDateEdit and QTimeEdit inherits from QAbstractSpinBox).
> 
> To complete the integration of the new widgets, we need signals emitted when the widgets are set using either setTime() or setDate(), i.e., change their values programmatically.
> In the current code, KTimeEdit has a signal timeChanged() and KDateEdit has dateChanged(), both emitted only when the user edits the widgets, but not when they are set programmatically.
> 
> So, last Wednesday dfaure suggested the following:
> 
> 1 - Rename the current timeChanged() to timeEdited() and dateChanged() to dateEdited();
> 2 - Update the uses of these signals across kdepim
> 3 - Add the new timeChanged() and dateChanged() and emit them in setTime and setDate, respectively.
> 
> With the listed changes, KTimeEdit and KDateEdit signals will follow what happens in Qt Edit classes (e.g. QLineEdit): a Changed signal whenever the content changes and an Edit signal emitted when the user edits the content.
> 
> The diff adds these new signals, document them and updates the current uses of {time,date}Changed() signals to {time,date}Edited().
> 
> Comments are welcome, as always =)
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdepim/incidenceeditor-ng/schedulingdialog.cpp 1178629 
>   /trunk/KDE/kdepim/kalarm/recurrenceedit.cpp 1178629 
>   /trunk/KDE/kdepim/libkdepim/kdateedit.h 1178629 
>   /trunk/KDE/kdepim/libkdepim/kdateedit.cpp 1178629 
>   /trunk/KDE/kdepim/libkdepim/kprefsdialog.cpp 1178629 
>   /trunk/KDE/kdepim/libkdepim/ktimeedit.h 1178629 
>   /trunk/KDE/kdepim/libkdepim/ktimeedit.cpp 1178629 
> 
> Diff: http://svn.reviewboard.kde.org/r/5432/diff
> 
> 
> Testing
> -------
> 
> KDEPIM builds successfully with the patch applied on top of r1178629. Applications affected by the changes working in the same way they did without the patch.
> 
> 
> Thanks,
> 
> Anselmo
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list