Re: Review Request: kickoff: save recent applications list on every change to it

2012-07-04 Thread Andriy Gapon
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo 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 zeitgeist. > > > > i've cc'd Trever on this because he may have something to s

Re: Review Request: kickoff: save recent applications list on every change to it

2012-07-04 Thread Andriy Gapon
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: > > plasma/desktop/applets/kickoff/core/recentapplications.cpp, line 42 > > > > > > holding on to groups like this is a bit dangerous; you need to be able > > to

Re: Review Request: kickoff: save recent applications list on every change to it

2012-07-04 Thread Andriy Gapon
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ --- (Updated July 4, 2012, 7:57 a.m.) Review request for Plasma and Trever Fis

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Seif Lotfy
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo 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 zeitgeist. > > > > i've cc'd Trever on this because he may have something to s

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Ivan Čukić
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo 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 zeitgeist. > > > > i've cc'd Trever on this because he may have something to s

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Trever Fischer
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo 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 zeitgeist. > > > > i've cc'd Trever on this because he may have something to s

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Trever Fischer
On Thu, 2012-05-31 at 16:45 +0200, Mark wrote: > On Thu, May 31, 2012 at 3:36 PM, Trever Fischer > wrote: > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105112/ > > > On May 31st, 2012, 10:09 a.m.,

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Mark
On Thu, May 31, 2012 at 3:36 PM, Trever Fischer wrote: >This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105112/ > > On May 31st, 2012, 10:09 a.m., *Aaron J. Seigo* wrote: > > the real fubar here is that it stores this information internally in it

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Aaron J. Seigo
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo 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 zeitgeist. > > > > i've cc'd Trever on this because he may have something to s

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Trever Fischer
> On May 31, 2012, 10:09 a.m., Aaron J. Seigo 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 zeitgeist. > > > > i've cc'd Trever on this because he may have something to s

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/#review14293 --- the real fubar here is that it stores this information internal

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ --- (Updated May 31, 2012, 10:08 a.m.) Review request for Plasma and Trever Fi

Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Andriy Gapon
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ --- Review request for Plasma. Description --- Currently recent applicati