Review Request: Add KIO::UDSEntry::UDS_ICON_OVERLAYS

David Faure faure at kde.org
Thu Apr 8 12:31:46 BST 2010


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

Ship it!


Looks good. Nice how new features can be added with only 2 lines of code :-)

- David


On 2010-04-08 09:56:15, Sebastian Trueg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3517/
> -----------------------------------------------------------
> 
> (Updated 2010-04-08 09:56:15)
> 
> 
> Review request for kdelibs and David Faure.
> 
> 
> Summary
> -------
> 
> KIO slaves should be able to add icon overlays. This patch adds exactly that in a straight-forward manner. The first slave to use it is my media:/ KIO slave which will follow this patch.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdelibs/kio/kio/kfileitem.cpp 1112475 
>   trunk/KDE/kdelibs/kio/kio/udsentry.h 1112475 
> 
> Diff: http://reviewboard.kde.org/r/3517/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sebastian
> 
>





More information about the kde-core-devel mailing list