Review Request 123280: Kickoff - Properly register recent applications right after startup

[David Edmundson]

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

Ship it!


Ship It!

- David Edmundson


On April 16, 2015, 1:09 p.m., Anthony Vital wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123280/
> -----------------------------------------------------------
> 
> (Updated April 16, 2015, 1:09 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> This does two things:
> - Instantiate the RecentlyUsedModel at startup, otherwise it's not available
> until the History tab is opened and this leads to some mixing in the recent
> apps list/the list not being updated until then.~~
> 
> - Move the registration of the service handler to the launcher, so that the
> apps launched from the Favorites tab are registered without having to open
> the Applications tab first.
> 
> 
> Diffs
> -----
> 
>   applets/kickoff/core/applicationmodel.cpp 9af58a36d5c3349ad1f2bffd0f7f93eebd99663e 
>   applets/kickoff/core/recentapplications.cpp 6c60fedcca1b0818179171a2d65c36ed3dd66787 
>   applets/kickoff/core/urlitemlauncher.cpp 58a6c20cec60ede54ea55d927064eaf8af9b4f46 
> 
> Diff: https://git.reviewboard.kde.org/r/123280/diff/
> 
> 
> Testing
> -------
> 
> Launched applications from Favorites and Applications tab without opening the History tab first, items are added correctly.
> 
> 
> Thanks,
> 
> Anthony Vital
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150416/2bc21622/attachment.html>