Set the "dialog-information" role for the sound being played.

* hildon/hildon-sound.c (hildon_play_system_sound): Set the
"dialog-information" role for the sound being played.

diff --git a/ChangeLog b/ChangeLog
index 7c932ad..0cbbb3c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2009-05-08  Claudio Saavedra  <csaavedra@igalia.com>
+
+       * hildon/hildon-sound.c (hildon_play_system_sound): Set the
+       "dialog-information" role for the sound being played.
+
 2009-05-06  Alberto Garcia  <agarcia@igalia.com>
 
        * hildon/hildon-app-menu.c (hildon_app_menu_init):

diff --git a/hildon/hildon-sound.c b/hildon/hildon-sound.c
index 1bbc264..980d65f 100644 (file)
--- a/hildon/hildon-sound.c
+++ b/hildon/hildon-sound.c
@@ -25,6 +25,9 @@
 /**
  * SECTION:hildon-sound
  * @short_description: libcanberra-based utility function for playing a sound.
+ *
+ * Please note that this method is only provided for backwards compatibility,
+ * but we highly recommend you to use canberra-gtk directly instead.
  * 
  */
 
@@ -81,7 +84,11 @@ hildon_ca_context_get (void)
  * @sample: sound file to play
  * 
  * Plays the given sample using libcanberra.
- * Volume level is received from gconf. 
+ * Volume level is received from gconf.
+ *
+ * This method sets the "dialog-information" role for the sound played,
+ * so you need to keep this into account when using it. For any purpose, it
+ * is highly recommended that you use canberra-gtk instead of this method.
  */
 void 
 hildon_play_system_sound(const gchar *sample)
@@ -120,6 +127,7 @@ hildon_play_system_sound(const gchar *sample)
 
     ca_proplist_create(&pl);
     ca_proplist_sets(pl, CA_PROP_MEDIA_FILENAME, sample);
+    ca_proplist_sets(pl, CA_PROP_MEDIA_ROLE, "dialog-information");
     ca_proplist_setf(pl, CA_PROP_CANBERRA_VOLUME, "%f", volume);
 
     ret = ca_context_play_full(ca_con, 0, pl, NULL, NULL);