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

Proposed by Danielle Foré
Status: Merged
Approved by: Cody Garver
Approved revision: 49
Merged at revision: 49
Proposed branch: lp:~wingpanel-devs/wingpanel-indicator-sound/symoblic-icons
Merge into: lp:~wingpanel-devs/wingpanel-indicator-sound/trunk
Diff against target: 28 lines (+5/-5)
1 file modified
src/Indicator.vala (+5/-5)
To merge this branch: bzr merge lp:~wingpanel-devs/wingpanel-indicator-sound/symoblic-icons
Reviewer Review Type Date Requested Status
WingPanel Devs Pending
Review via email: mp+275610@code.launchpad.net

Commit message

use proper -symbolic audio icons and not notification-audio icons

To post a comment you must log in.
49. By Danielle Foré

ditch notification-audio-* icons

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Indicator.vala'
2--- src/Indicator.vala 2015-09-13 09:07:04 +0000
3+++ src/Indicator.vala 2015-10-23 23:56:59 +0000
4@@ -243,13 +243,13 @@
5 if (open == false && this.notification != null && v >= -0.05 && v <= (this.max_volume + 0.05)) {
6 string icon;
7 if (v <= 0.0) {
8- icon = "notification-audio-volume-off";
9+ icon = "audio-volume-muted-symbolic";
10 } else if (v <= 0.3) {
11- icon = "notification-audio-volume-low";
12+ icon = "audio-volume-low-symbolic";
13 } else if (v <= 0.7) {
14- icon = "notification-audio-volume-medium";
15+ icon = "audio-volume-medium-symbolic";
16 } else {
17- icon = "notification-audio-volume-high";
18+ icon = "audio-volume-high-symbolic";
19 }
20
21 this.notification.update ("indicator-sound", "", icon);
22@@ -410,4 +410,4 @@
23
24 var indicator = new Sound.Indicator ();
25 return indicator;
26-}
27\ No newline at end of file
28+}

Subscribers

People subscribed via source and target branches

to all changes: