Wingpanel Sound Indicator

Merge lp:~elementary-pantheon/wingpanel-indicator-sound/default-app-icon into lp:~wingpanel-devs/wingpanel-indicator-sound/trunk

  1. default-app-icon
  2. Merge into trunk
Proposed by Danielle Foré
Status: Merged
Approved by: Djax
Approved revision: 96
Merged at revision: 96
Proposed branch: lp:~elementary-pantheon/wingpanel-indicator-sound/default-app-icon
Merge into: lp:~wingpanel-devs/wingpanel-indicator-sound/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
src/Widgets/MprisGui.vala (+1/-1)
To merge this branch: bzr merge lp:~elementary-pantheon/wingpanel-indicator-sound/default-app-icon
Reviewer Review Type Date Requested Status
WingPanel Devs Pending
Review via email: mp+300222@code.launchpad.net

Commit message

Use application-default-icon when MPRIS app icon is unknown

Description of the change

emblem-music is not an application icon. Instead, use the application fallback icon as it was designed

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Widgets/MprisGui.vala'
2--- src/Widgets/MprisGui.vala 2016-03-20 03:13:35 +0000
3+++ src/Widgets/MprisGui.vala 2016-07-15 16:52:57 +0000
4@@ -66,7 +66,7 @@
5 }
6 }
7 if (app_icon == null) {
8- app_icon = new ThemedIcon ("emblem-music-symbolic");
9+ app_icon = new ThemedIcon ("application-default-icon");
10 }
11
12 create_ui ();

Subscribers

People subscribed via source and target branches

to all changes: