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
=== modified file 'plugins/media-keys/gsd-media-keys-manager.c'
--- plugins/media-keys/gsd-media-keys-manager.c 2015-08-25 09:19:53 +0000
+++ plugins/media-keys/gsd-media-keys-manager.c 2015-10-12 16:14:50 +0000
@@ -257,6 +257,22 @@
257 NULL257 NULL
258};258};
259259
260static const char *audio_volume_icons[] = {
261 "audio-volume-muted-symbolic",
262 "audio-volume-low-symbolic",
263 "audio-volume-medium-symbolic",
264 "audio-volume-high-symbolic",
265 NULL
266};
267
268static const char *mic_volume_icons[] = {
269 "microphone-sensitivity-muted-symbolic",
270 "microphone-sensitivity-low-symbolic",
271 "microphone-sensitivity-medium-symbolic",
272 "microphone-sensitivity-high-symbolic",
273 NULL
274};
275
260static const char *brightness_icons[] = {276static const char *brightness_icons[] = {
261 "notification-display-brightness-off",277 "notification-display-brightness-off",
262 "notification-display-brightness-low",278 "notification-display-brightness-low",
@@ -343,10 +359,13 @@
343static gboolean359static gboolean
344ubuntu_osd_notification_show_volume (GsdMediaKeysManager *manager,360ubuntu_osd_notification_show_volume (GsdMediaKeysManager *manager,
345 gint value,361 gint value,
346 gboolean muted)362 gboolean muted,
363 gboolean is_mic)
347{364{
365 const char **icons_name = is_mic ? mic_volume_icons : volume_icons;
366
348 return ubuntu_osd_do_notification (&manager->priv->volume_notification,367 return ubuntu_osd_do_notification (&manager->priv->volume_notification,
349 "volume", value, muted, volume_icons);368 "volume", value, muted, icons_name);
350}369}
351370
352static gboolean371static gboolean
@@ -606,20 +625,6 @@
606 gboolean muted,625 gboolean muted,
607 int volume)626 int volume)
608{627{
609 static const char *icon_names[] = {
610 "audio-volume-muted-symbolic",
611 "audio-volume-low-symbolic",
612 "audio-volume-medium-symbolic",
613 "audio-volume-high-symbolic",
614 NULL
615 };
616 static const char *mic_icon_names[] = {
617 "microphone-sensitivity-muted-symbolic",
618 "microphone-sensitivity-low-symbolic",
619 "microphone-sensitivity-medium-symbolic",
620 "microphone-sensitivity-high-symbolic",
621 NULL
622 };
623 int n;628 int n;
624629
625 if (muted) {630 if (muted) {
@@ -635,9 +640,9 @@
635 }640 }
636641
637 if (is_mic)642 if (is_mic)
638 return mic_icon_names[n];643 return mic_volume_icons[n];
639 else644 else
640 return icon_names[n];645 return audio_volume_icons[n];
641}646}
642647
643static gboolean648static gboolean
@@ -1337,7 +1342,7 @@
1337 const GvcMixerStreamPort *port;1342 const GvcMixerStreamPort *port;
1338 const char *icon;1343 const char *icon;
13391344
1340 if (ubuntu_osd_notification_show_volume (manager, vol, muted))1345 if (ubuntu_osd_notification_show_volume (manager, vol, muted, !GVC_IS_MIXER_SINK (stream)))
1341 goto done;1346 goto done;
13421347
1343 vol = CLAMP (vol, 0, 100);1348 vol = CLAMP (vol, 0, 100);

Subscribers

People subscribed via source and target branches