----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103103/#review8351 -----------------------------------------------------------
This patch modifies the behaviour, maybe it is better if you change KDateComboBox::NoneKeyword to KDateComboBox::NoNoneKeyword And adapt the if accordingly? This way there is no behaviour change at all and makes "old" users still work. - Albert Astals Cid 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 > >