[Icon Widget] Don't use plasma theme for icon
ClosedPublic
Actions

Authored by broulik on Jun 2 2016, 9:56 AM.

Details

Reviewers

davidedmundson
andreask

Group Reviewers

Plasma

Commits

Summary

This avoids picking a black and white icon like it would for KMail.
Neither Task Manager nor Kickoff use plasma theme icons for applications.

Test Plan

KMail has proper icon now

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.

broulik retitled this revision from to [Icon Widget] Don't use plasma theme for icon.

broulik updated this object.

broulik edited the test plan for this revision. (Show Details)

broulik set the repository for this revision to R120 Plasma Workspace.

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

This revision is now accepted and ready to land.Jun 2 2016, 9:57 AM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			applets/icon/package/contents/ui/main.qml (1 line)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	4158			Jun 2 2016, 9:55 AM	★	★
Diff 2	4159	294f8e8	rPLASMAWORKSPACEb434a0f05ed93e40c50c78b31f16fd7afafe1ff3	Jun 2 2016, 9:58 AM	★	★