Review Request: Implements Plasma::TabBar::setTabHighlighted

[Aaron J. Seigo]

-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110504/92dc19ae/attachment.htm