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