> On 2009-11-07 09:51:49, Marco Martin wrote: > > seems good, just that small issue.
this has been committed, hasn't it? please mark as submitted - Beat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2088/#review2967 ----------------------------------------------------------- On 2009-11-29 00:56:27, John Layt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/2088/ > ----------------------------------------------------------- > > (Updated 2009-11-29 00:56:27) > > > Review request for Plasma. > > > Summary > ------- > > The CalendarTable widget does not correctly display in RTL mode, the cells in > the table go LTR instead. The numbers in the cells are controlled by code > and not the widget layout, so we need to handle this the code. > > This patch fixes the day numbers, weekday names, and week numbers to go RTL, > see screenshots for example. It doesn't move the Week Numbers column from > the left to the right as this is part of the SVG, we would need to flip the > SVG along it's vertical axis to do so and shuffle everything about. Is this > possible or too much effort? > > Also rename a couple of variables for clarity seeing as I now understand what > they do. > > Patch against trunk, I'll backport to 4.3 if all OK. Note we may want to > rewrite some of the CalendarTable in 4.5 for a cleaner implementation. > > cc to Dotan to confirm this looks OK and to confirm it's not a problem if we > can't move the Week Number column. > > > This addresses bug 182976. > https://bugs.kde.org/show_bug.cgi?id=182976 > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/libs/plasmaclock/calendartable.cpp 1055882 > > Diff: http://reviewboard.kde.org/r/2088/diff > > > Testing > ------- > > Tested using 'plasmoidviewer --reverse calendar' > > > Screenshots > ----------- > > Left to Right mode > http://reviewboard.kde.org/r/2088/s/253/ > Right to Left mode > http://reviewboard.kde.org/r/2088/s/254/ > > > Thanks, > > John > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
