Review Request: Improve display of week number in calendar applet

[John Layt]

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


You can test RTL by running "plasmoidviewer calendar -reverse".  Alternative calendars are even easier, just right-click to configure.  Anyway, they look OK.  I might tweak the 'most days' formula.  Thanks!

- John


On 2010-06-21 06:07:40, Alain Boyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4419/
> -----------------------------------------------------------
> 
> (Updated 2010-06-21 06:07:40)
> 
> 
> 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/b4bf26a3/attachment.htm