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 June 21st, 2010, 6:07 a.m., Alain Boyer wrote:
|
Review request for Plasma.
By Alain Boyer.
Updated 2010-06-21 06:07:40
Description
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.
|
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.
|
Diffs
- /trunk/KDE/kdebase/workspace/libs/plasmaclock/calendartable.cpp (1140389)
View Diff
|