Review Request: Implements Plasma::TabBar::setTabHighlighted
Matthias Fuchs
mat69 at gmx.net
Sun May 1 15:31:31 CEST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101264/
-----------------------------------------------------------
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/
Thanks,
Matthias
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110501/87423cb2/attachment.htm
More information about the Plasma-devel
mailing list