Don't use plasma theme icons for stream/device icons
Needs RevisionPublic

Authored by broulik on Jun 26 2017, 10:00 AM.

Details

Reviewers
drosca
hein
Group Reviewers
Plasma
Test Plan

VLC has a Plasma icon now and we get a mix of apps and plasma icons. We typically don't use plasma theme for app-heavy things like task manager, launcher, etc

Diff Detail

Repository
R115 Plasma Audio Volume Applet
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Jun 26 2017, 10:00 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 26 2017, 10:00 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
hein accepted this revision.Jun 26 2017, 10:02 AM
This revision is now accepted and ready to land.Jun 26 2017, 10:02 AM
drosca requested changes to this revision.Jun 27 2017, 8:58 AM

If we want this, then only for applications tab (streams), because there is no reason not to use Plasma theme for device icons (those are standard freedesktop icon names).

This revision now requires changes to proceed.Jun 27 2017, 8:58 AM