Review Request: Improve display of week number in calendar applet

[Alain Boyer]

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

Review request for Plasma.


Summary
-------

This patch is a temporary workaround for bugs 238116 and 224344. I am calling it temporary since it is my understanding that the drawing of the calendar applet will be worked on and improved for 4.6. Is this correct?

Basically, only one week number is shown when there is not enough room to display two week numbers in the calendar applet (two week numbers are displayed when the first day of the week is not Monday). Instead of simply displaying the first week number, an attempt is made to display the week number that has the most amount of days in the calendar row.


Diffs
-----

  /trunk/KDE/kdebase/workspace/libs/plasmaclock/calendartable.cpp 1140389 

Diff: http://reviewboard.kde.org/r/4419/diff


Testing
-------

Tested with Georgian calendar and seems to give correct behavior no matter which day of the week is set as the first day of the week.

Needs testing with other calendars and RTL layouts.


Thanks,

Alain

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100621/554ff530/attachment.htm