Review Request: Provide extra options for date keyword display in KDateComboBox

[David Jarvie]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103103/#review8265
-----------------------------------------------------------


Any comments on this? - it would be good to get it into 4.8. It is essentially a bug fix, since without this the keyword facility is unusable for some applications. 

- David Jarvie


On Nov. 10, 2011, 12:36 a.m., David Jarvie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103103/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2011, 12:36 a.m.)
> 
> 
> Review request for kdelibs and John Layt.
> 
> 
> Description
> -------
> 
> KDateComboBox provides the option to display a list of date keywords with the date picker popup. These keywords currently show dates in the past, present and future, together with a "no date" item. The "no date" item is a particular problem since not only is it redundant - since there is already a button in the line edit to clear the date value - but I suspect that many applications will not accept a blank date as valid input.
> 
> This patch adds two new enum values to allow applications to select which of these date keywords are displayed: one enum value shows the "no date" item which is now not shown by default, and another enum value hides dates in the past.
> 
> Currently in KAlarm, which I maintain, the display of date keywords is disabled because dates in the past and blank dates are not valid values to enter, and it would be confusing to users to offer them as options. This patch will make it possible to display only present and future date keywords, which will enable it to make use of date keywords.
> 
> 
> Diffs
> -----
> 
>   kdeui/widgets/kdatecombobox.h 63bf52f 
>   kdeui/widgets/kdatecombobox.cpp fc239bc 
> 
> Diff: http://git.reviewboard.kde.org/r/103103/diff/diff
> 
> 
> Testing
> -------
> 
> Tested in KAlarm.
> 
> 
> Thanks,
> 
> David Jarvie
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111117/46318c91/attachment.htm>