After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 246431 - Add the possibility to preview the sound used for mail notification
Add the possibility to preview the sound used for mail notification
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
2.8.x (obsolete)
Other All
: Normal enhancement
: Future
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
: 250404 (view as bug list)
Depends on: 492702
Blocks:
 
 
Reported: 2003-07-16 00:58 UTC by Mariano Suárez-Alvarez
Modified: 2013-09-13 00:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for sound preview (5.78 KB, patch)
2006-01-17 11:35 UTC, Johnny Jacob
none Details | Review
patch without gnome_file_entry* API. (5.73 KB, patch)
2007-08-22 07:55 UTC, anand v m
needs-work Details | Review
updated patch. (5.14 KB, patch)
2007-08-26 09:38 UTC, anand v m
none Details | Review

Description Mariano Suárez-Alvarez 2003-07-16 00:58:28 UTC
I'd be useful/nice if the Settings dialog would allow to preview (prehead?)
the sound file.
Comment 1 André Klapper 2005-07-16 19:09:06 UTC
still missing in 2.3.5.1, reassigning to harish.
i want this. :-)
Comment 2 André Klapper 2005-11-21 12:24:06 UTC
*** Bug 250404 has been marked as a duplicate of this bug. ***
Comment 3 Johnny Jacob 2006-01-10 20:51:32 UTC
patch on the list.
Comment 4 André Klapper 2006-01-11 09:09:12 UTC
cool, looking forward to this
Comment 5 Johnny Jacob 2006-01-17 11:35:10 UTC
Created attachment 57524 [details] [review]
Patch for sound preview
Comment 6 André Klapper 2006-02-18 17:13:42 UTC
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".
Comment 7 Srinivasa Ragavan 2006-07-11 16:15:22 UTC
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.
Comment 8 Srinivasa Ragavan 2006-07-11 16:15:45 UTC
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.
Comment 9 André Klapper 2006-07-14 17:42:59 UTC
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?
Comment 10 anand v m 2007-08-22 07:55:15 UTC
Created attachment 94091 [details] [review]
patch without gnome_file_entry*  API.
Comment 11 Johnny Jacob 2007-08-23 12:58:56 UTC
Patch doesn't apply cleanly. Needs Work. 
Anand , test before you upload your patch :) 
Comment 12 anand v m 2007-08-26 09:38:49 UTC
Created attachment 94350 [details] [review]
updated patch.
Comment 13 Srinivasa Ragavan 2007-08-27 04:58:41 UTC
Johhny, can you review this? Candidate for 2.13.1
Comment 14 Srinivasa Ragavan 2007-12-04 12:21:14 UTC
Johnny, Can you get this in asap?
Comment 15 Johnny Jacob 2007-12-05 05:35:20 UTC
Srini, This has to be reworked / merged with the new mail notify work. Marking this bugs as depends on Bug 492702 .
Comment 16 Johnny Jacob 2007-12-15 12:05:40 UTC
Closing. Patch went along with bug 492702. (trunk : svn rev 34702)