Review Request 126578: [Kickoff/Kicker] Add context menu actions for search results
Thomas Pfeiffer
colomar at autistici.org
Wed Dec 30 22:01:32 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126578/#review90361
-----------------------------------------------------------
Looks great, no objections!
- Thomas Pfeiffer
On Dec. 30, 2015, 7:09 p.m., Kai Uwe Broulik wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126578/
> -----------------------------------------------------------
>
> (Updated Dec. 30, 2015, 7:09 p.m.)
>
>
> Review request for Plasma, KDE Usability and Eike Hein.
>
>
> Repository: plasma-desktop
>
>
> Description
> -------
>
> This adds the context menu actions you also get in the favorites and applications view to search results.
>
> Usability: This enables this for Kickoff, Kicker and Application Dashboard.
>
> I did not add the "Edit application" context menu as search results are somewhat conceptually detached from the application menu structure.
>
>
> Diffs
> -----
>
> applets/kicker/package/contents/ui/main.qml 5defa75
> applets/kicker/plugin/actionlist.h 58dd6dd
> applets/kicker/plugin/actionlist.cpp 6d73434
> applets/kicker/plugin/appentry.cpp 61fd8a9
> applets/kicker/plugin/runnermatchesmodel.cpp b6b1b0f
> applets/kicker/plugin/runnermodel.h 4c3f212
> applets/kicker/plugin/runnermodel.cpp 6496816
> applets/kickoff/package/contents/ui/Kickoff.qml 8a8c9c5
>
> Diff: https://git.reviewboard.kde.org/r/126578/diff/
>
>
> Testing
> -------
>
> Steam favorite: Add to desktop, panel, task manager work, "Screenshots" jump list action works
>
> Searched for "steam": Add to desktop, panel, task manager work, "Screenshots" jump list action works
>
>
> File Attachments
> ----------------
>
> Search result menu
> https://git.reviewboard.kde.org/media/uploaded/files/2015/12/30/2e00c239-3dcc-4253-9c0c-0563a11de9b2__steaminsearch.jpg
>
>
> Thanks,
>
> Kai Uwe Broulik
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151230/ecd99b76/attachment.html>
More information about the Plasma-devel
mailing list