Differential D4396

Specify NoDisplay type
ClosedPublic
Actions

Authored by apol on Feb 1 2017, 5:23 PM.

Details

Reviewers
davidedmundson
mart
Group Reviewers
Frameworks
Plasma
Commits
R242:6659517bce2d: Specify NoDisplay type
Summary

Otherwise toBool doesn't work.
I didn't see it called anywhere as property("NoDisplay") == "true", so it
shouldn't be a problem like similar cases before

Test Plan

See upcoming patches

Diff Detail

Repository
R242 Plasma Framework (Library)
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
apol updated this revision to Diff 10825.Feb 1 2017, 5:23 PM
apol retitled this revision from to Specify NoDisplay type.
apol updated this object.
apol edited the test plan for this revision. (Show Details)
apol added reviewers: Plasma, Frameworks, mart, davidedmundson.
Restricted Application added projects: Plasma, Frameworks. · View Herald TranscriptFeb 1 2017, 5:23 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Feb 2 2017, 3:19 PM
mart edited edge metadata.
This revision is now accepted and ready to land.Feb 2 2017, 3:19 PM
Closed by commit R242:6659517bce2d: Specify NoDisplay type (authored by apol). · Explain WhyFeb 2 2017, 4:26 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 10825

View Options

src/plasma/data/servicetypes/plasma-applet.desktop

Loading...