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

List:       koffice-devel
Subject:    Re: [patch] Allow sound effects without visual effects
From:       Thomas Zander <zander () kde ! org>
Date:       2004-05-28 16:26:04
Message-ID: 200405281826.09178.zander () kde ! org
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Friday 28 May 2004 14:08, Brad Hards wrote:
> > Right now, slide transition sound effects only work if you also have a
> > visual effect enabled. Thats not very intuitive (three "WhatsThis"
> > entries of warning :-), and quite unnecessary.

Looks good to me; but I would appreciate a KPresenter guru to look at as 
well.
One thingy; you use the clicked() signal and then call it since your 
programattically setting it does not call the changed method itself.
What about using the toggled() slot instead, so the setEnabled() will call 
the disappearSoundEffectChanged() for you?

>      connect( disappearSoundEffect, SIGNAL( clicked() ), this, SLOT( 
> disappearSoundEffectChanged() ) );
...
> +    disappearSoundEffect->setEnabled( disappear->isChecked() );
> +    disappearSoundEffectChanged();

- -- 
Thomas Zander
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFAt2ggCojCW6H2z/QRAjFEAKCOlcYiCTvaC1os5otAJUMC0VWKlQCfcW7S
z+MOa5TGMmw8GtNy25SB5y8=
=ifiE
-----END PGP SIGNATURE-----
_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel

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