[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request: Bug with Plasma::Delegate text painter
From: "Jesse L. Zamora" <xtremek2008 () aim ! com>
Date: 2008-11-20 20:44:42
Message-ID: 200811201544.42386.xtremek2008 () aim ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
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! :-)
[Attachment #5 (text/html)]
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" \
"http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" \
content="1" /><style type="text/css">p, li { white-space: pre-wrap; \
}</style></head><body style=" font-family:'Sans Serif'; font-size:10pt; \
font-weight:400; font-style:normal;">On Thursday 20 November 2008 12:45:00 Aaron J. \
Seigo wrote:<br> > On Thursday 20 November 2008, Jesse L. Zamora wrote:<br>
> > > in the delegate option->direction<br>
> > > otherwise QApplication::layoutDirection() as aaron said<br>
> > > and you can do a quick text lauching plasma with the --reverse \
option,<br> > > > would look a bit weird since the text will still be in a \
left to right<br> > > > language but gives an idea of how the layout \
looks<br> > > ><br>
> > > Cheers,<br>
> > > Marco Martin<br>
> ><br>
> > Yikes, I tried to do that (I ran plasma with --reverse),<br>
><br>
> you can also do `plasmoidviewer -reverse launcher` for quicker turn-around<br>
> times.<br>
><br>
> > and Kickoff looks<br>
> > TERRIBLE! Is that what it's going to look like with a right to left<br>
> > language?<br>
><br>
> terrible? the only problems i saw were:<br>
><br>
> * the arrows in the page switch view (used on the Applications tab) aren't<br>
> being fully painted with the hover color<br>
><br>
> * the text sometimes runs outside the selection box, but that's also<br>
> visible in LTR too (which is what you were fixing =)<br>
><br>
> what problems did you see?<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; \
margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; \
-qt-user-state:0;"><br></p>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! \
:-)</p></body></html>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic