Merge lp:~elementary-pantheon/wingpanel-indicator-sound/default-app-icon into lp:~wingpanel-devs/wingpanel-indicator-sound/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
=== modified file 'src/Widgets/MprisGui.vala'
--- src/Widgets/MprisGui.vala 2016-03-20 03:13:35 +0000
+++ src/Widgets/MprisGui.vala 2016-07-15 16:52:57 +0000
@@ -66,7 +66,7 @@
66 }66 }
67 }67 }
68 if (app_icon == null) {68 if (app_icon == null) {
69 app_icon = new ThemedIcon ("emblem-music-symbolic");69 app_icon = new ThemedIcon ("application-default-icon");
70 }70 }
7171
72 create_ui ();72 create_ui ();

Subscribers

People subscribed via source and target branches

to all changes: