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

List:       kmail-devel
Subject:    Re: [PATCH] composing html messages
From:       Don Sanders <sanders () kde ! org>
Date:       2004-02-10 6:01:31
Message-ID: 200402101601.31995.sanders () kde ! org
[Download RAW message or body]

On Sunday 08 February 2004 21:22, Edwin Schepers wrote:
> Hi,
> Finally it looks I have something workable.
> It's not final: indentation, deleting some kDebug's and naming has
> to be changed.
> Could you (someone) review ?
> If this is the way to go, and when it looks quite stable, can I
> commit (after the changes to make it final)?

Great! The basic functionality of creating multipart/alternative 
messages is working well now.

I think just some relatively minor clean ups and the patch can go in. 
Specifically removing the DEBUG define and commented out code, 
disabling the html toolbar by default and if possible resolving the 
too large line spacing issue and the patch should be ok.

Great work.

Thanks,
Don.
_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel
[prev in list] [next in list] [prev in thread] [next in thread]