Don't show entries with NoDisplay=true with the applauncher containmentaction
ClosedPublic
Actions

Authored by fvogt on Feb 19 2019, 2:10 PM.

Details

Reviewers

Group Reviewers

Commits

Summary

Currently NoDisplay services are shown, which doesn't make much sense.

Test Plan

Installed and looked through the menu, no excessive or duplicate items anymore.

Repository

R120 Plasma Workspace

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

fvogt requested review of this revision.Feb 19 2019, 2:10 PM

This revision is now accepted and ready to land.Feb 21 2019, 7:42 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			containmentactions/applauncher/launch.cpp (2 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	52068	ff34471		Feb 19 2019, 2:10 PM	★	★
Diff 2	52243	f6bf548	R120:8192bd3aec27a77bb1ab903bb928ffe0a2f4c517	Feb 21 2019, 9:19 PM	★	★

Commit	Tree	Parents	Author	Summary	Date
57da4bb179de	ead2f5080a47	ff34471aecd3	Fabian Vogt	Don't show entries with NoDisplay=true with the applauncher containmentaction (Show More…)	Feb 19 2019, 1:21 PM