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 127751 - only use one soundlist for balsa
only use one soundlist for balsa
Status: RESOLVED FIXED
Product: balsa
Classification: Other
Component: general
2.0.x
Other All
: Normal minor
: ---
Assigned To: Balsa Maintainers
Balsa Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-11-23 18:02 UTC by dobradovic
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Consolidate soundlists (578 bytes, patch)
2004-11-17 00:00 UTC, Craig Routledge 		none Details | Review

Description dobradovic 2003-11-23 18:02:23 UTC 
Currently, balsa installs two soundlists with only one event in each one.
I'd suggest to remove the email.soundlist and merge its event into
balsa.soundlist. I see only confusion in having two very distinct sections
in control-center.
Comment 1 Pawel Salek 2003-11-23 22:56:17 UTC 
A provided patch will most likely be accepted (some new sounds would
be appreciated too!)
Comment 2 dobradovic 2003-11-23 23:18:30 UTC 
If a patch handles removed files as well, ok. But you'll still need to
delete them via cvs, right? As it is not a that big deal... :)

But I'm not sure about what to do. Does Balsa need a sound for
starting up? Anyone known who doesn't turn it off after the fifth
startup? *g*
I think, every app could have such a sound, but they don't.

And the new-mail-sound is cool, I like it.
Comment 3 Craig Routledge 2004-11-17 00:00:39 UTC 
Created attachment 33863 [details] [review]
Consolidate soundlists

In addition to the patch, sounds/email.soundlist.in needs a "cvs remove"

The translations are still picked up correctly, but the references in the po
files are strange.  I don't know why they point to things like
"balsa.soundlist.in.h:2" instead of "balsa.soundlist.in:6"  Anybody know about
this?
Comment 4 Pawel Salek 2004-11-17 21:48:45 UTC 
Patch commited - thanks!