Differential D19157

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
hein
Group Reviewers
Plasma
Commits
R120:7816978b5120: Don't show entries with NoDisplay=true with the applauncher containmentaction
R120:8192bd3aec27: Don't show entries with NoDisplay=true with the applauncher containmentaction
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.

Diff Detail

Repository
R120 Plasma Workspace
Branch
nodisplay
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 8539
Build 8557: arc lint + arc unit
fvogt created this revision.Feb 19 2019, 2:10 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 19 2019, 2:10 PM
fvogt requested review of this revision.Feb 19 2019, 2:10 PM
hein accepted this revision.Feb 21 2019, 7:42 PM
This revision is now accepted and ready to land.Feb 21 2019, 7:42 PM
Closed by commit R120:8192bd3aec27: Don't show entries with NoDisplay=true with the applauncher containmentaction (authored by fvogt). · Explain WhyFeb 21 2019, 9:19 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 52068

View Options

containmentactions/applauncher/launch.cpp

Loading...