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

List:       kopete-devel
Subject:    [Kopete-devel] Re: kdenonbeta/kopete/kopete
From:       James Grant <topace () lightbox ! org>
Date:       2003-06-17 19:43:06
[Download RAW message or body]

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

I actually started a "FirstRunWizard" about a month or so ago, then my time 
got cut off.. I didnt get very far into it...  Im hoping that over the next 
several weeks my time will fee up a bit and I can get back at it.

Cheers,
  James


On June 17, 2003 02:21 pm, Martijn Klingens wrote:
> CVS commit by mklingens:
>
> When starting Kopete for the very first time show the prefs dialog. Not
> perfect, but better than nothing.
>
> Whoever wants to contribute to Kopete and feels bored is free to implement
> the FIXMEs that I added :)
>
>
>   M +18 -5     kopete.cpp   1.142
>
>
> --- kdenonbeta/kopete/kopete/kopete.cpp  #1.141:1.142
> @@ -34,4 +34,5 @@
>  #include "kopetewindow.h"
>  #include "pluginconfig.h"
> +#include "preferencesdialog.h"
>
>  #ifdef HAVE_CONFIG_H
> @@ -108,4 +109,5 @@ void Kopete::slotLoadPlugins()
>
>          QStringList modules;
> +        bool showConfigDialog = false;
>
>          if( config->hasKey( "Plugins" ) )
> @@ -115,9 +117,17 @@ void Kopete::slotLoadPlugins()
>          else
>          {
> -                // Ups! the user does not have plugins selected.
> -                // TODO: show "first time" wizard and let user decide
> which modules to load -                modules.append("history.desktop");
> -                modules.append("contactnotes.desktop");
> -                // Other modules to load for the first time?
> +                // No plugins specified. Show the config dialog.
> +                // FIXME: Although it's a bit stupid it is theoretically
> possible that a user +                //        explicitly configured
> Kopete to not load plugins on startup. In this +                //       
> case we don't want this dialog. We need some other config setting +        
>        //        like a bool hasRunKopeteBefore or so to trigger the
> loading of the +                //        wizard. Maybe using the last run
> version number is more useful even +                //        as it also
> allows for other features. - Martijn +                // FIXME: Of course
> this is not a too-good GUI because a first-timer would need +              
>  //        some kind of "welcome" dialog or wizard. But for now it's better
> than +                //        nothing at all. - Martijn
> +                // FIXME: Possibly we need to influence the
> showConfigDialog bool based on the +                //        command line
> arguments processed below. But how exactly? - Martijn +               
> showConfigDialog = true;
>          }
>
> @@ -162,4 +172,6 @@ void Kopete::slotLoadPlugins()
>          }
>
> +        if( showConfigDialog )
> +                PreferencesDialog::preferencesDialog()->show();
>  }

- -- 
James Grant
topace@lightbox.org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE+729KV7tGVeQW9pcRAp2gAJwKB/wbi15rFtFrwjxjWOHnDXsEyACgnF8Q
OG/QS/MNMA4S2Jkz4hn7OPk=
=iGq0
-----END PGP SIGNATURE-----

_______________________________________________
Kopete-devel mailing list
Kopete-devel@mail.kde.org
http://mail.kde.org/mailman/listinfo/kopete-devel

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