Review Request: Use Config to Provide Activity Icons

[Aaron J. Seigo]

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


"nepomuk works differently" should not be a reason for this particular change.

the question is whether to store the icons in nepomuk or not. if they are stored in nepomuk, then the information stays with the activities. if they are stored outside of nepomuk, then when we want to do things like migrate activities it becomes messier and we lose visibility on the connection between the image and the activity.

- Aaron J. Seigo


On May 22, 2012, 3:12 a.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105008/
> -----------------------------------------------------------
> 
> (Updated May 22, 2012, 3:12 a.m.)
> 
> 
> Review request for Plasma and Ivan Čukić.
> 
> 
> Description
> -------
> 
> Since we can't trust Nepomuk anymore now that ontologies are loaded asynchronously, we better implement the approach we have in master.
> 
> 
> This addresses bug 298684.
>     http://bugs.kde.org/show_bug.cgi?id=298684
> 
> 
> Diffs
> -----
> 
>   service/ActivityManager.cpp 8927c43 
>   service/ActivityManager_p.h 3ea1d8a 
> 
> Diff: http://git.reviewboard.kde.org/r/105008/diff/
> 
> 
> Testing
> -------
> 
> Log out and back in. Before this patch, icons would probably load later, or not load at all. Now icons are displayed right away when desktop is loaded.
> 
> 
> Thanks,
> 
> David Narváez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120522/4ce1a06c/attachment.html>