Review Request: Bug with Plasma::Delegate text painter

[Jesse L. Zamora]

On Thursday 20 November 2008 12:45:00 Aaron J. Seigo wrote:
> 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?

Actually, the code I had written made it look terrible. I have now fixed that, 
so now it correctly has gradients in both ltr and rtl layouts! :-)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20081120/32554ce9/attachment.htm