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

List:       kde-pim
Subject:    Re: [Kde-pim]
From:       Thomas McGuire <mcguire () kde ! org>
Date:       2009-05-05 13:03:29
Message-ID: 200905051503.29925.mcguire () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Hi,

On Tuesday 05 May 2009 12:30:18 Volker Krause wrote:
> On Tuesday 05 May 2009 01:16:23 Ingo Klöcker wrote:
> > On Tuesday 05 May 2009, Thomas McGuire wrote:
> > > now that KPIMTextEdit is basically complete, I wanted to support
> > > images in signatures.
> > > The signature editor is in kpimidentities, so kpimidentities needs to
> > > link to kpimtextedit so that the signature editor there can support
> > > images.
> > >
> > > Problem is, kpimtextedit already links to kpimidentities, because it
> > > has some functions to insert signatures into the text edit.
> >
> > Why do those functions exist in kpimtextedit? Wouldn't it be sufficient
> > to have a method insertSignature() which is called by somebody (e.g.
> > KMail) who can link against both libraries?
>
> right, getting rid of these looks like the best way to solve this. Can't we
> move those to kpimidentities instead?

I think moving those methods to the Signature class in kpimidentities might 
indeed be the best idea, I'll try that.
If that doesn't work I'll move them back to kmeditor in libkdepim, but that 
would mean Mailody can't use these methods.

Regards,
Thomas

["signature.asc" (application/pgp-signature)]

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/

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