Merge lp:~seb128/unity-settings-daemon/sru-mic-mute-icon into lp:unity-settings-daemon/14.04

Proposed by Sebastien Bacher
Status: Merged
Approved by: Iain Lane
Approved revision: 4047
Merged at revision: 4047
Proposed branch: lp:~seb128/unity-settings-daemon/sru-mic-mute-icon
Merge into: lp:unity-settings-daemon/14.04
Diff against target: 84 lines (+24/-19)
1 file modified
plugins/media-keys/gsd-media-keys-manager.c (+24/-19)
To merge this branch: bzr merge lp:~seb128/unity-settings-daemon/sru-mic-mute-icon
Reviewer Review Type Date Requested Status
Iain Lane Approve
Review via email: mp+274164@code.launchpad.net

Commit message

show correct microphone mute icon

Description of the change

show correct microphone mute icon

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

ok

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/media-keys/gsd-media-keys-manager.c'
2--- plugins/media-keys/gsd-media-keys-manager.c 2015-08-25 09:19:53 +0000
3+++ plugins/media-keys/gsd-media-keys-manager.c 2015-10-12 16:14:50 +0000
4@@ -257,6 +257,22 @@
5 NULL
6 };
7
8+static const char *audio_volume_icons[] = {
9+ "audio-volume-muted-symbolic",
10+ "audio-volume-low-symbolic",
11+ "audio-volume-medium-symbolic",
12+ "audio-volume-high-symbolic",
13+ NULL
14+};
15+
16+static const char *mic_volume_icons[] = {
17+ "microphone-sensitivity-muted-symbolic",
18+ "microphone-sensitivity-low-symbolic",
19+ "microphone-sensitivity-medium-symbolic",
20+ "microphone-sensitivity-high-symbolic",
21+ NULL
22+};
23+
24 static const char *brightness_icons[] = {
25 "notification-display-brightness-off",
26 "notification-display-brightness-low",
27@@ -343,10 +359,13 @@
28 static gboolean
29 ubuntu_osd_notification_show_volume (GsdMediaKeysManager *manager,
30 gint value,
31- gboolean muted)
32+ gboolean muted,
33+ gboolean is_mic)
34 {
35+ const char **icons_name = is_mic ? mic_volume_icons : volume_icons;
36+
37 return ubuntu_osd_do_notification (&manager->priv->volume_notification,
38- "volume", value, muted, volume_icons);
39+ "volume", value, muted, icons_name);
40 }
41
42 static gboolean
43@@ -606,20 +625,6 @@
44 gboolean muted,
45 int volume)
46 {
47- static const char *icon_names[] = {
48- "audio-volume-muted-symbolic",
49- "audio-volume-low-symbolic",
50- "audio-volume-medium-symbolic",
51- "audio-volume-high-symbolic",
52- NULL
53- };
54- static const char *mic_icon_names[] = {
55- "microphone-sensitivity-muted-symbolic",
56- "microphone-sensitivity-low-symbolic",
57- "microphone-sensitivity-medium-symbolic",
58- "microphone-sensitivity-high-symbolic",
59- NULL
60- };
61 int n;
62
63 if (muted) {
64@@ -635,9 +640,9 @@
65 }
66
67 if (is_mic)
68- return mic_icon_names[n];
69+ return mic_volume_icons[n];
70 else
71- return icon_names[n];
72+ return audio_volume_icons[n];
73 }
74
75 static gboolean
76@@ -1337,7 +1342,7 @@
77 const GvcMixerStreamPort *port;
78 const char *icon;
79
80- if (ubuntu_osd_notification_show_volume (manager, vol, muted))
81+ if (ubuntu_osd_notification_show_volume (manager, vol, muted, !GVC_IS_MIXER_SINK (stream)))
82 goto done;
83
84 vol = CLAMP (vol, 0, 100);

Subscribers

People subscribed via source and target branches