Review Request: Bug with Plasma::Delegate text painter

[Aaron J. Seigo]

On Thursday 20 November 2008, Jesse L. Zamora wrote:
> > in the delegate option->direction
> > otherwise QApplication::layoutDirection() as aaron said
> > and you can do a quick text lauching plasma with the --reverse option,
> > would look a bit weird since the text will still be in a left to right
> > language but gives an idea of how the layout looks
> >
> > Cheers,
> > Marco Martin
>
> Yikes, I tried to do that (I ran plasma with --reverse), 

you can also do `plasmoidviewer -reverse launcher` for quicker turn-around 
times.

> and Kickoff looks
> TERRIBLE! Is that what it's going to look like with a right to left
> language?

terrible? the only problems i saw were:

* the arrows in the page switch view (used on the Applications tab) aren't 
being fully painted with the hover color

* the text sometimes runs outside the selection box, but that's also visible 
in LTR too (which is what you were fixing =)

what problems did you see?

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20081120/56346722/attachment.sig