From kde-core-devel Wed Jan 30 01:43:24 2002 From: Waldo Bastian Date: Wed, 30 Jan 2002 01:43:24 +0000 To: kde-core-devel Subject: Re: PATCH - Disable All Sounds button in System Notifications X-MARC-Message: https://marc.info/?l=kde-core-devel&m=101235509405985 On Tuesday 29 January 2002 05:38 pm, Neil Stevens wrote: > On Tuesday January 29, 2002 04:25, Waldo Bastian wrote: > > Hm.. what about changing the text on the pushbutton then? So that we > > toggle between "disable all sounds" and "enable all sounds"? > > > > That would solve both problems nicely. We can check for > > !event->soundfile.isEmpty() to decide which sounds should be part of > > "all sounds". > > I like that idea. Here's the second patch attempt using that idea. Looks good :-) Cheers, Waldo --=20 bastian@kde.org | SuSE Labs KDE Developer | bastian@suse.com