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)
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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 10852

View Options

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

Loading...