kdepim/korganizer

[Bo Thorsen]

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

On Tuesday 18 May 2004 01:13, Reinhold Kainhofer wrote:
> CVS commit by kainhofe:
>
> Revert my commit to get rid of the deprecated setActiveWindow. There is
> simply no function that replaces the whole functionality of
> setActiveWindow!

I think this is the fourth time I have seen this checked in and reverted 
again. This doesn't work, and even without a crystal ball I'll bet it's 
not the last time we've seen this.

Cornelius, could you give a summary of the problem here? You're the one 
that seem to understand what the problem is. With the problem described, 
perhaps someone can come up with a solution. Staying with a deprecated 
method does not seem like a viable solution to me.

Either setActiveWindow must be undeprecated again, or an alternative must 
be made. Or perhaps there already is a solution that we just don't know 
about.

Bo.

"I'm sure a solution will present itself" - Qui-Gon Jinn.

> --- kdepim/korganizer/actionmanager.cpp  #1.74:1.75
> @@ -593,5 +593,5 @@ void ActionManager::file_open()
>    KOrg::MainWindow *korg=ActionManager::findInstance(url);
>    if ((0 != korg)&&(korg != mMainWindow)) {
> -    KWin::forceActiveWindow(korg->topLevelWidget()->winId());
> +    KWin::setActiveWindow(korg->topLevelWidget()->winId());
>      return;
>    }
> @@ -613,5 +613,5 @@ void ActionManager::file_openRecent(cons
>      if ((0 != korg)&&(korg != mMainWindow)) {
>                    // already open in a different windows, activate
> that one -     
> KWin::forceActiveWindow(korg->topLevelWidget()->winId()); +     
> KWin::setActiveWindow(korg->topLevelWidget()->winId()); return;
>      }
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

iD8DBQFAqa/KmT99lwfUS5IRAoFvAJ9CQe2R5PqXXc+rqkqcNgkA4+4MhgCg1eno
B95l3CiM2eMgDe5FiHuWUhE=
=rnEI
-----END PGP SIGNATURE-----