Review Request 125675: Use the new KOverlayIconPlugin interface from KIO::Widgets

[Emmanuel Pescosta]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125675/#review87070
-----------------------------------------------------------



src/kitemviews/kfileitemmodelrolesupdater.cpp (line 52)
<https://git.reviewboard.kde.org/r/125675/#comment59846>

    Please remove this version check and increase the min. KF version to 5.16 instead. Such checks only lead to problems and increase the amount of testing dramatically, so only add them if they are unavoidable. (KF 5.16 will be released before the beta release of Applications 15.12)



src/kitemviews/kfileitemmodelrolesupdater.cpp (lines 140 - 150)
<https://git.reviewboard.kde.org/r/125675/#comment59847>

    IMHO this and all the foreach overlay plugin loops should also go into KIO hidden behind
    a nice service api (getOverlays and overlaysChanged signal). Clients are usually not interested where the overlays come from and where the necessary plugins are located ...
    
    This avoids duplicated code when other clients want to make use of the overlay infrastructure and makes it easier to extend in future.


- Emmanuel Pescosta


On Oct. 19, 2015, 1:14 p.m., Olivier Goffart wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125675/
> -----------------------------------------------------------
> 
> (Updated Oct. 19, 2015, 1:14 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> Depends on, https://git.reviewboard.kde.org/r/125436
> 
> Replaces https://git.reviewboard.kde.org/r/125136/
> 
> 
> Diffs
> -----
> 
>   src/kitemviews/kfileitemmodelrolesupdater.cpp 14e3015 
>   src/CMakeLists.txt 413260b 
>   src/kitemviews/kfileitemmodelrolesupdater.h 6e7559f 
> 
> Diff: https://git.reviewboard.kde.org/r/125675/diff/
> 
> 
> Testing
> -------
> 
> Tested with owncloud plugin.
> 
> 
> Thanks,
> 
> Olivier Goffart
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20151019/a873ac61/attachment.htm>