This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/105112/

On May 31st, 2012, 10:09 a.m., Aaron J. Sei= go wrote:

the real =
fubar here is that it stores this information internally in its own config =
file. this really ought to be stored/retrieved from nepomuk and/or zeitgeis=
t.

i've cc'd Trever on this because he may have something to say about=
 that as well.

On May 31st, 2012, 1:37 p.m., Trever Fischer wrote:

I actuall=
y just recently patched Dragon to do so, and it took very few lines: http:/=
/quickgit.kde.org/index.php?p=3Ddragon.git&a=3Dcommitdiff&h=3D92fb6=
296e424dc829e0c5cc541aa3581856d2098

Since Kickoff uses QAbstractItemModels, switching things to use a QZeitgeis=
t::LogModel should be trivial. Alternatively, implementing the RecentApplic=
ations class to use Zeitgeist can be trivial as well, and would seem like t=
he easiest way to do things.

On May 31st, 2012, 2:43 p.m., Aaron J. Seigo wrote:

if we cou=
ld dump RecentApplications that'd be great. there's not much reason=
 to go through a local class just to get to another library (qzeitgeist) cl=
ass. then the LogModel can be used in the RecentlyUsedModel .. =


in fact, if KRecentDocument in kdelibs/kio/kfile/ were made to use zeitgeis=
t, i bet we could just drop the RecentlyUsedModel altogether, or at worst h=
ave it as a thin layer around LogModel (don't know what LogModel provid=
es, so I can't really offer a concrete suggestion there). that sounds l=
ike frameworks 5 work, though, so we'll have to just hold off on that f=
or kickoff .. but would seem to be the natural progression and something th=
at could implemented immediately in the frameworks branch in any case...

On May 31st, 2012, 4:36 p.m., Trever Fischer wrote:

logmodel.=
h: http://quickgit.kde.org/index.php?p=3Dlibqzeitgeist.git&a=3Dblob&=
;h=3D2ef6bfac0ab917ed5508da64b3a9d1a9290e65e6&hb=3Da58d1caaa953b514b7bd=
1697c44aebf75b11829d&f=3Dsrc%2Flogmodel.h

It provides a bunch of shortcuts for access to the underlying event object =
data, or you can access the event in its entirety.

What it doesn't provide that the recentlyusedmodel provides is the imme=
diate ability to remove events. You'd need to grab the event and then a=
sk a Log object to delete it from the database. I am considering adding tha=
t API to the 0.10 release.

On May 31st, 2012, 4:45 p.m., Ivan =C4=8Cuki=C4=87 wrote:

Another t=
hing for these types of models that we *need* to start adding is the suppor=
t for different results based on the activity.

Has Zeitgeist moved in that direction as promised before (in Randa)?

This is sup=
ported. Just attach the id of the activity to the event.origin. You can the=
n query Zeitgeist with event.origin set to the activity id and it will retu=
rn only results based on events that happened in that activity

- Seif

On May 31st, 2012, 10:08 a.m., Andriy Gapon wrote:

Review request for Plasma and Trever Fischer.

By Andriy Gapon.

Updated May 31, 2012, 10:08 a.m.

Descripti= on

Currently recent applications list in kickoff is saved only =
when kickoff gracefully exits.  This could be a minor annoyance when X/KDE/=
plasma crashes.  I think that saving the list on every update to it should =
be a good idea.  It should be a low overhead too, because the list changes =
only when a user launches an application via KDE.

Bugs: 206511

Diffs=

plasma/desktop/applets/kickoff/core/recentapplications.cpp (3e05389)

View Diff