Merge lp:~speakman/ubuntu/natty/padevchooser/drop-use_stock_gnome_icons.patch into lp:ubuntu/natty/padevchooser

Proposed by Daniel Nyström
Status: Rejected
Rejected by: Martin Pitt
Proposed branch: lp:~speakman/ubuntu/natty/padevchooser/drop-use_stock_gnome_icons.patch
Merge into: lp:ubuntu/natty/padevchooser
Diff against target: 117 lines (+6/-90)
3 files modified
debian/changelog (+6/-0)
debian/patches/series (+0/-1)
debian/patches/use_stock_gnome_icons.patch (+0/-89)
To merge this branch: bzr merge lp:~speakman/ubuntu/natty/padevchooser/drop-use_stock_gnome_icons.patch
Reviewer Review Type Date Requested Status
Martin Pitt Disapprove
Review via email: mp+59425@code.launchpad.net

Description of the change

Get icons back to padevchooser. Although they doesn't match Ubuntu default theme, there are at least some.

(Sorry for re-requesting - wrong commit email in previous version)

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Closing, see bug report. Thanks!

review: Disapprove

Unmerged revisions

8. By Daniel Nyström

Drop use_stock_gnome_icons.patch. Fixes (LP: #632468)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2009-04-16 18:36:50 +0000
3+++ debian/changelog 2011-04-28 20:59:53 +0000
4@@ -1,3 +1,9 @@
5+padevchooser (0.9.3-2ubuntu5) natty; urgency=low
6+
7+ * Drop use_stock_gnome_icons.patch. Fixes (LP: #632468)
8+
9+ -- Daniel Nyström <daniel@nystrom.st> Thu, 28 Apr 2011 22:22:05 +0200
10+
11 padevchooser (0.9.3-2ubuntu4) jaunty; urgency=low
12
13 * lp147305.patch: Don't crash when zeroconf devices disappear
14
15=== modified file 'debian/patches/series'
16--- debian/patches/series 2009-04-16 18:36:50 +0000
17+++ debian/patches/series 2011-04-28 20:59:53 +0000
18@@ -1,5 +1,4 @@
19 # This series applies on GIT commit 42cd6a27e16cacb8b33efd4105497c864ce4e8ff
20 statustray_svn24.patch
21-use_stock_gnome_icons.patch
22 desktop_hig_comment.patch
23 lp147305.patch
24
25=== removed file 'debian/patches/use_stock_gnome_icons.patch'
26--- debian/patches/use_stock_gnome_icons.patch 2008-03-21 14:31:04 +0000
27+++ debian/patches/use_stock_gnome_icons.patch 1970-01-01 00:00:00 +0000
28@@ -1,89 +0,0 @@
29-Use stock Gnome icons (instead of Tango icons)
30-
31-From: CJ van den Berg <cj@vdbonline.com>
32-
33-
34----
35-
36- src/padevchooser.c | 14 +++++++-------
37- src/padevchooser.desktop | 2 +-
38- src/padevchooser.glade | 2 +-
39- 3 files changed, 9 insertions(+), 9 deletions(-)
40-
41-Index: padevchooser-0.9.3/src/padevchooser.c
42-===================================================================
43---- padevchooser-0.9.3.orig/src/padevchooser.c 2008-03-21 15:13:46.000000000 -0500
44-+++ padevchooser-0.9.3/src/padevchooser.c 2008-03-21 15:14:49.000000000 -0500
45-@@ -162,13 +162,13 @@
46-
47- if (!notification) {
48- s = g_strdup_printf("<i>%s</i>\n%s", title, text);
49-- notification = notify_notification_new(title, s, "audio-card", GTK_WIDGET(tray_icon));
50-+ notification = notify_notification_new(title, s, "stock_line_in", GTK_WIDGET(tray_icon));
51- notify_notification_set_category(notification, "device.added");
52- notify_notification_set_urgency(notification, NOTIFY_URGENCY_LOW);
53- g_signal_connect_swapped(G_OBJECT(notification), "closed", G_CALLBACK(notification_closed), NULL);
54- } else {
55- s = g_strdup_printf("%s\n\n<i>%s</i>\n%s", last_events, title, text);
56-- notify_notification_update(notification, title, s, "audio-card");
57-+ notify_notification_update(notification, title, s, "stock_line_in");
58- }
59-
60- g_free(last_events);
61-@@ -525,7 +525,7 @@
62- tray_icon = gtk_status_icon_new();
63-
64- g_signal_connect_object(G_OBJECT(tray_icon), "activate", G_CALLBACK(tray_icon_on_click), tray_icon, 0);
65-- gtk_status_icon_set_from_icon_name(tray_icon, "audio-card");
66-+ gtk_status_icon_set_from_icon_name(tray_icon, "stock_line_in");
67- gtk_status_icon_set_tooltip(tray_icon, "PulseAudio Applet");
68- gtk_status_icon_set_visible(tray_icon, TRUE);
69-
70-@@ -583,9 +583,9 @@
71- append_default_device_menu_items(source_submenu, &no_sources_menu_item, &default_source_menu_item, &other_source_menu_item, source_default_cb, source_other_cb);
72- append_default_device_menu_items(server_submenu, &no_servers_menu_item, &default_server_menu_item, &other_server_menu_item, server_default_cb, server_other_cb);
73-
74-- append_submenu(menu, "Default S_erver", server_submenu, "network-wired");
75-- append_submenu(menu, "Default S_ink", sink_submenu, "audio-card");
76-- append_submenu(menu, "Default S_ource", source_submenu, "audio-input-microphone");
77-+ append_submenu(menu, "Default S_erver", server_submenu, "gnome-dev-ethernet");
78-+ append_submenu(menu, "Default S_ink", sink_submenu, "stock_headphones");
79-+ append_submenu(menu, "Default S_ource", source_submenu, "stock_mic");
80- gtk_menu_shell_append(GTK_MENU_SHELL(menu), gtk_separator_menu_item_new());
81-
82- item = append_menuitem(menu, "_Manager...", NULL);
83-@@ -593,7 +593,7 @@
84- g_free(c);
85- g_signal_connect(G_OBJECT(item), "activate", G_CALLBACK(start_manager_cb), NULL);
86-
87-- item = append_menuitem(menu, "_Volume Control...", "multimedia-volume-control");
88-+ item = append_menuitem(menu, "_Volume Control...", "stock_volume");
89- gtk_widget_set_sensitive(item, !!(c = g_find_program_in_path("pavucontrol")));
90- g_free(c);
91- g_signal_connect(G_OBJECT(item), "activate", G_CALLBACK(start_vucontrol_cb), NULL);
92-Index: padevchooser-0.9.3/src/padevchooser.desktop
93-===================================================================
94---- padevchooser-0.9.3.orig/src/padevchooser.desktop 2008-03-21 15:12:36.000000000 -0500
95-+++ padevchooser-0.9.3/src/padevchooser.desktop 2008-03-21 15:13:48.000000000 -0500
96-@@ -5,7 +5,7 @@
97- GenericName=Device Chooser
98- Comment=An Applet for Quick Device Changing
99- Exec=padevchooser
100--Icon=audio-card
101-+Icon=stock_line_in
102- StartupNotify=true
103- Type=Application
104- Categories=Application;AudioVideo;Audio;
105-Index: padevchooser-0.9.3/src/padevchooser.glade
106-===================================================================
107---- padevchooser-0.9.3.orig/src/padevchooser.glade 2008-03-21 15:12:36.000000000 -0500
108-+++ padevchooser-0.9.3/src/padevchooser.glade 2008-03-21 15:13:48.000000000 -0500
109-@@ -76,7 +76,7 @@
110- <widget class="GtkImage" id="image1">
111- <property name="visible">True</property>
112- <property name="icon_size">6</property>
113-- <property name="icon_name">audio-card</property>
114-+ <property name="icon_name">stock_sound</property>
115- <property name="xalign">0.5</property>
116- <property name="yalign">0.5</property>
117- <property name="xpad">0</property>

Subscribers

People subscribed via source and target branches