Merge lp:~sergiusens/account-polld/sound_from_xdg_data into lp:~phablet-team/account-polld/trunk

Proposed by Sergio Schvezov
Status: Merged
Approved by: Sergio Schvezov
Approved revision: 28
Merged at revision: 26
Proposed branch: lp:~sergiusens/account-polld/sound_from_xdg_data
Merge into: lp:~phablet-team/account-polld/trunk
Prerequisite: lp:~sergiusens/account-polld/reset_poll
Diff against target: 13 lines (+2/-1)
1 file modified
plugins/plugins.go (+2/-1)
To merge this branch: bzr merge lp:~sergiusens/account-polld/sound_from_xdg_data
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Roberto Alsina (community) Approve
Review via email: mp+228527@code.launchpad.net

Commit message

Set the default sound to a path relative to an XDG_DATA_DIR as the push client searches within those paths.

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
28. By Sergio Schvezov

Fix typo

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/plugins.go'
--- plugins/plugins.go 2014-07-28 16:02:34 +0000
+++ plugins/plugins.go 2014-07-28 16:02:35 +0000
@@ -145,7 +145,8 @@
145145
146// DefaultSound returns the path to the default sound for a Notification146// DefaultSound returns the path to the default sound for a Notification
147func DefaultSound() string {147func DefaultSound() string {
148 return "/usr/share/sounds/ubuntu/notifications/Slick.ogg"148 // path is searched within XDG_DATA_DIRS
149 return "sounds/ubuntu/notifications/Slick.ogg"
149}150}
150151
151// DefaultVibration returns a Vibrate with the default vibration152// DefaultVibration returns a Vibrate with the default vibration

Subscribers

People subscribed via source and target branches