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 671446 - Two sounds are played when choosing new alert sound after choosing Default
Two sounds are played when choosing new alert sound after choosing Default
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Sound
3.3.x
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
3.10
: 724610 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-03-06 08:01 UTC by Walter Garcia-Fontes
Modified: 2015-03-16 10:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
sound: use row-activated to listen to alert sound changes (3.87 KB, patch)
2015-03-15 00:04 UTC, Cosimo Cecchi
accepted-commit_now Details | Review

Description Walter Garcia-Fontes 2012-03-06 08:01:00 UTC
Ubuntu 12.04 Beta 1
Unity: System settings -> Hardware -> Sound

1. Open the Sound settings dialog
2. Choose the Sound Effects tab
3. Select 'Default'
4. Select 'Bark' (or any option other than 'Default')

You will hear the default sound and the bark sound simultaneously.
Comment 1 Bastien Nocera 2012-03-07 11:50:27 UTC
What are the version numbers for gnome-settings-daemon and gnome-control-center? Are they still bodging a different theme in there?
Comment 2 Walter Garcia-Fontes 2012-03-07 12:15:17 UTC
gnome-settings 3.3.91

gnome-settings 3.3.91

The theme name shown in org.gnome.desktop.sound is "__custom"
Comment 3 Bastien Nocera 2012-03-07 12:16:29 UTC
Want to double-check those version numbers? :)
Comment 4 Bastien Nocera 2012-03-07 12:21:27 UTC
And I bet that the bug is caused by the patch "98_default_sound_theme.patch", as they don't correctly change the default value in gsettings-desktop-schemas".

Let me know when you've tested and reproduced the problem with either an unpatched version of the sound panel, or a patched version of gsettings-desktop-schemas.
Comment 5 Sebastien Bacher 2012-03-07 12:59:40 UTC
I didn't check, I will have a look but out this patch we have a gsettings override

"[org.gnome.desktop.sound]
theme-name = 'ubuntu'"

and /etc/gtk-3.0/settings.ini

"gtk-sound-theme-name = ubuntu"
Comment 6 Andreas Trepczik 2012-04-26 12:49:45 UTC
I can reproduce the reported problem with gnome-control-center and gnome-settings-daemon at 3.4.1 on Arch Linux, which only lightly patches these packages for better non-systemd support. It already happened with GNOME 3.2 and the problem also occurs with Fedora 17 Beta.
Comment 7 Cosimo Cecchi 2015-03-14 23:00:22 UTC
*** Bug 724610 has been marked as a duplicate of this bug. ***
Comment 8 Cosimo Cecchi 2015-03-14 23:01:16 UTC
I can reproduce this 100% in g-c-c 3.15.91 on Fedora 22 alpha.
Comment 9 Cosimo Cecchi 2015-03-15 00:04:54 UTC
Created attachment 299430 [details] [review]
sound: use row-activated to listen to alert sound changes

We currently install two signal handlers for button-press-event and
selection-changed. As the former also calls the latter, the two interact
in non-obvious ways that can lead to two alerts being played at the same
time under certain circumstances.
Avoid this problem by setting activate-on-single-click on the tree view
and listening to the row-activated signal to drive previews and model
changes.
Comment 10 Bastien Nocera 2015-03-15 07:28:38 UTC
Comment on attachment 299430 [details] [review]
sound: use row-activated to listen to alert sound changes

If that fixes the bug, looks good.
Comment 11 Cosimo Cecchi 2015-03-15 19:29:05 UTC
Attachment 299430 [details] pushed as 6137c60 - sound: use row-activated to listen to alert sound changes

Thanks, pushed.
Comment 12 Bastien Nocera 2015-03-16 10:02:00 UTC
Cherry-picked in gnome-3-8 and up, as well.