GNOME Bugzilla – Bug 246431
Add the possibility to preview the sound used for mail notification
Last modified: 2013-09-13 00:48:48 UTC
I'd be useful/nice if the Settings dialog would allow to preview (prehead?) the sound file.
still missing in 2.3.5.1, reassigning to harish. i want this. :-)
*** Bug 250404 has been marked as a duplicate of this bug. ***
patch on the list.
cool, looking forward to this
Created attachment 57524 [details] [review] Patch for sound preview
applied the patch, it works fine. i hope to see this in 2.7, but i would like to have a translator comment for "_Play", something like "Play this sound file".
Agreed Andre's comment. Also gnome_file_entry* api? I remember this cant be used? Some patches from SUN were there to replace/fix this. Dont remember fully the issue.
jony: "gnome_file_entry_gtk_entry is deprecated and should not be used in newly-written code.", see http://developer.gnome.org/doc/API/2.0/libgnomeui/GnomeFileEntry.html can you write an updated patch so we can get it in before UI freeze?
Created attachment 94091 [details] [review] patch without gnome_file_entry* API.
Patch doesn't apply cleanly. Needs Work. Anand , test before you upload your patch :)
Created attachment 94350 [details] [review] updated patch.
Johhny, can you review this? Candidate for 2.13.1
Johnny, Can you get this in asap?
Srini, This has to be reworked / merged with the new mail notify work. Marking this bugs as depends on Bug 492702 .
Closing. Patch went along with bug 492702. (trunk : svn rev 34702)