----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/274/#review266 -----------------------------------------------------------
/trunk/KDE/kdelibs/plasma/delegate.cpp <http://reviewboard.vidsolbach.de/r/274/#comment216> there's no point in painting this if the text fits, right? that would avoid painting this for every single item. - Aaron On 2008-11-20 13:06:28, Jesse Zamora wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.vidsolbach.de/r/274/ > ----------------------------------------------------------- > > (Updated 2008-11-20 13:06:28) > > > Review request for Plasma. > > > Summary > ------- > > I have fixed the bug in Plasma::Delegate (in the screenshot) where long > strings run out of the item rects and make it look ugly. It adds a gradient > to make it look better (as in the 2nd screenshot). Please test & review this > patch so maybe it can go into trunk for KDE 4.2. > > Xtreme Kommander > > > Diffs > ----- > > /trunk/KDE/kdelibs/plasma/delegate.cpp > > Diff: http://reviewboard.vidsolbach.de/r/274/diff > > > Testing > ------- > > > Screenshots > ----------- > > Plasma delegate text painting bug > http://reviewboard.vidsolbach.de/r/274/s/94/ > Fixed bug in Plasma::Delegate > http://reviewboard.vidsolbach.de/r/274/s/95/ > > > Thanks, > > Jesse > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
