Merge lp:~xapantu/wingpanel-indicator-sound/trunk into lp:~wingpanel-devs/wingpanel-indicator-sound/trunk

Proposed by xapantu
Status: Merged
Merged at revision: 10
Proposed branch: lp:~xapantu/wingpanel-indicator-sound/trunk
Merge into: lp:~wingpanel-devs/wingpanel-indicator-sound/trunk
Diff against target: 29 lines (+3/-3)
1 file modified
src/Widgets/MprisGui.vala (+3/-3)
To merge this branch: bzr merge lp:~xapantu/wingpanel-indicator-sound/trunk
Reviewer Review Type Date Requested Status
WingPanel Devs Pending
Review via email: mp+261193@code.launchpad.net
To post a comment you must log in.
10. By xapantu

initialize background once app_icon is initialized

11. By xapantu

Remove a call using background which is not initialized

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 2015-06-04 10:48:06 +0000
3+++ src/Widgets/MprisGui.vala 2015-06-05 10:27:56 +0000
4@@ -53,7 +53,6 @@
5 ainfo = new DesktopAppInfo (client.player.desktop_entry + ".desktop");
6 if (ainfo != null) {
7 app_icon = ainfo.get_icon ();
8- background = new Gtk.Image.from_gicon (app_icon, Gtk.IconSize.DIALOG);
9 app_name = ainfo.get_display_name ();
10 if (app_name == "")
11 app_name = ainfo.get_name ();
12@@ -61,8 +60,9 @@
13 }
14 if (app_icon == null) {
15 app_icon = new ThemedIcon ("emblem-music-symbolic");
16- background.set_from_icon_name ("emblem-music-symbolic", Gtk.IconSize.DIALOG);
17 }
18+
19+ background = new Gtk.Image.from_gicon (app_icon, Gtk.IconSize.DIALOG);
20
21 background.margin_start = 6;
22 background.margin_end = 6;
23@@ -328,4 +328,4 @@
24
25 return Gdk.pixbuf_get_from_surface (surface, 0, 0, mask_size, mask_size);
26 }
27-}
28\ No newline at end of file
29+}

Subscribers

People subscribed via source and target branches

to all changes: