[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: PATCH - Disable All Sounds button in System Notifications
From:       Waldo Bastian <bastian () kde ! org>
Date:       2002-01-30 1:43:24
[Download RAW message or body]

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

[prev in list] [next in list] [prev in thread] [next in thread]