[PATCH] BR57204 mailto: urls don't work when kontact is running.

[Ingo Klöcker]

On Monday 15 December 2003 13:55, Waldo Bastian wrote:
> The attached patch is a concept patch to fix BR57204.
> You must make sure to have an up to date
> kdelibs/kdecore/kcmdlineargs.{cpp,h}
>
> It still needs a bit of tweaking because it currently creates a new
> kmail window when kontact is started without arguments.
>
> On the bright side, with this patch applied you can now use "kontact
> bastian at kde.org" to complain about that :-)
>
> Other open issues are:
> * Startup notification may need some attention.
> * kwin does not put the composer window on top when started as
> "kontact bastian at kde.org"

Well, that's probably due to the fact that Luboš forced us to use 
KWin::activateWindow instead of KWin::forceActiveWindow (or the 
deprecated KWin::setActiveWindow). I'm sure he can tell us what to 
do. ;-)

> * KMail command line options may not get translated correctly.

Okay, this will fix the KMail case. Any idea what we should do when we 
want to call KAddressBook from KMail (e.g. if the user selects Open in 
Addressbook from the context menu for email addresses? Currently we use 
startServiceByDesktopName("kaddressbook") which starts KAddressBook 
externally if the address book part hasn't been loaded yet in Kontact 
and in case the address book part has already been loaded nothing 
happens (except for the startup notification which blinks until it 
times out). Both is wrong because in both cases the Contacts part 
should be brought to the front in Kontact. The second case can easily 
be fixed by first checking if we can contact kaddressbook via DCOP. But 
what about the first case? Will a similar solution as the one for kmail 
also work for kaddressbook?

Regards,
Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: signature
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20031215/9e821635/attachment.sig>