----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101264/#review3102 -----------------------------------------------------------
Ship it! looks fine; a few minor tweaks but nothing that stands in the way of committing this. cheers :) plasma/private/nativetabbar.cpp <http://git.reviewboard.kde.org/r/101264/#comment2633> it's a private class; private membesr are unecessary. :) plasma/private/nativetabbar.cpp <http://git.reviewboard.kde.org/r/101264/#comment2634> watch the whitespace => count()) { - Aaron J. On May 2, 2011, 10:25 a.m., Matthias Fuchs wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101264/ > ----------------------------------------------------------- > > (Updated May 2, 2011, 10:25 a.m.) > > > Review request for Plasma, Aaron J. Seigo and Marco Martin. > > > Summary > ------- > > Adds a setTabHighlighted and a isTabHighligted method to Plasma::TabBar. > When a tab is highlighted its icon will be drawn with 50% opacity and the > text will use Theme::HighlightColor. > > So far I am not overly happy with the result. > Marco suggested to blur the area behind the icon, though I am not sure how to > do that. Another option I thought off is to either draw the button highlight > effect at the area of highlighted tabs or around its icon. > What do you think of that? > > > Diffs > ----- > > plasma/private/nativetabbar.cpp 916deed > plasma/private/nativetabbar_p.h 665d99f > plasma/widgets/tabbar.h eb7a2aa > plasma/widgets/tabbar.cpp 16da3b9 > > Diff: http://git.reviewboard.kde.org/r/101264/diff > > > Testing > ------- > > > Screenshots > ----------- > > could be nicer :/ > http://git.reviewboard.kde.org/r/101264/s/152/ > > http://git.reviewboard.kde.org/r/101264/s/153/ > > > Thanks, > > Matthias > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
