[Kicker] Add "Edit application" and "Manage application" entry to service runner results
AbandonedPublic
Actions

Authored by broulik on Aug 28 2017, 12:02 PM.

Details

Reviewers

hein

Group Reviewers

Plasma

Summary

This makes application search results consistent with applications in the menu.

The ifdef'd AppStream logic is completely moved into the Kicker namespace and MenuEntryEditor is turned into a proper singleton.

BUG: 384036
FIXED-IN: 5.11.0

Test Plan

Edit Application and Manage %1 is still properly shown for applications. Separators look sane with either or both of them shown.
The entries are properly shown and functional now on service runner application results

Repository

R119 Plasma Desktop

Lint

Lint Skipped

Unit

Unit Tests Skipped

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

Slight cleanup

Duplicate of D7567.

I actually set you as reviewer on that one via arc, but somehow it dropped all subscribers, fuck. Sorry about the unnecessary work.

			Path	Packages
M			applets/kicker/plugin/actionlist.h (5 lines)
M			applets/kicker/plugin/actionlist.cpp (80 lines)
M			applets/kicker/plugin/appentry.h (1 line)
M			applets/kicker/plugin/appentry.cpp (64 lines)
M			applets/kicker/plugin/menuentryeditor.h (6 lines)
M			applets/kicker/plugin/menuentryeditor.cpp (6 lines)
M			applets/kicker/plugin/runnermatchesmodel.cpp (8 lines)