> On May 22, 2012, 12:52 p.m., Aaron J. Seigo wrote: > > "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. > > David Narváez wrote: > This review request has nothing to do with storing icons in Nepomuk or > not. It has to do with what is returned to a client when it requests an icon > for an activity - if we wait for Nepomuk to start and then return the icon > there or just use the config file which already has the info. Icons are still > stored in Nepomuk and sync'ed later in the code once Nepomuk is available.
let me rephrase for clarity: storing activity data (esp duplicate data) outside of nepomuk all to work around annoyances (like the one your patch addresses) or to placate people who insist they won't enable nepomuk makes no sense at this point. it would be far more sensible to ensure that this data is consistently and quickly available from nepomuk than keeping another cache of this information outside of nepomuk. - Aaron J. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105008/#review14047 ----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
