Patch for kdatepicker with weekStartDay != Monday
Reinhold Kainhofer
reinhold at kainhofer.com
Thu Oct 16 10:42:44 BST 2003
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
yesterday I took a look at bug 65588 and wrote the attached patch for the date
picker. So far, if the week start day was not monday, but the month started
on a monday, a click selected the date a week ahead.
My patch cleans up the position <=> date conversion by introducing two
(non-virtual, thus binary compatible) functions
int posFromDate( const QDate &date );
QDate dateFromPos( int pos );
which do all the offset handling (due to week start day). Also, now the date
picker behaves exactly the same no matter which weekStart day is selected (so
far, there were checks for weekStartDay()<4 in the code, which didn't make
sense to me...). There is always at least one day and at most 8 days of the
previous month visible.
Now, before I commit, can somebody with more knowledge of calendar systems and
locale settings take a look at it and tell me that it is correct?
If it's not not correct, I'll have to rework some code in korganizer, too...
Okay to apply?
- --
- ------------------------------------------------------------------
Reinhold Kainhofer, Graz, Austria
email: reinhold at kainhofer.com, http://reinhold.kainhofer.com/
* Mathematics Department, Graz University of Technology
* Deltasoft mathematics, http://www.deltasoft.at/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (GNU/Linux)
iD8DBQE/jmgXTqjEwhXvPN0RAnJVAKCwkFioCemcrlEFI5wkmOndx4x/rACgv8fo
OGgZSsSHvnGIWpxATT/z9LU=
=OCDU
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 2003-10-15_kdeui_WeekStart_KDateTable_Bug65588.patch
Type: text/x-diff
Size: 7963 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20031016/1f670169/attachment.patch>
More information about the kde-core-devel
mailing list