Review Request: Make IconTask use Plasma::Theme'd "close" button

Diego Casella polentino911 at gmail.com
Wed Nov 9 10:22:39 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103092/
-----------------------------------------------------------

(Updated Nov. 9, 2011, 10:22 a.m.)


Review request for Plasma, Aaron J. Seigo and Craig Drummond.


Description
-------

Simple patch which makes IconTask look more consistent with Plasma Theme.
It renders the "close" button using Plasma::Theme, and fallback to the standard window-close icon if nothing better is available :)


Diffs
-----

  applets/icontasks/tooltips/windowpreview.cpp 3e0c865 

Diff: http://git.reviewboard.kde.org/r/103092/diff/diff


Testing
-------

Compiles and works as expected (see pictures below).


Screenshots (updated)
-----------

ToolTip with themed "close" button
  http://git.reviewboard.kde.org/r/103092/s/329/
ToolTip with default "close" icon
  http://git.reviewboard.kde.org/r/103092/s/330/


Thanks,

Diego Casella

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111109/9e48b263/attachment.html>


More information about the Plasma-devel mailing list