[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-pim
Subject: Re: [Kde-pim] Mailreader status and RFC
From: Mantia Andras <amantia () kde ! org>
Date: 2009-09-11 6:18:37
Message-ID: 20090911071414.24467.qmail () ktown ! kde ! org
[Download RAW message or body]
Hi,
Thomas McGuire wrote:
> Also, some stuff of the mailreader has to be used by other libs/apps, like
> the objecttreeparser, which is needed in various places in KMail, and the
> stringutils functions.
Stringutils should be no problem to share (once we decide where to put
them). But the objecttreeparser needs to be decoupled from the private
MailViewer class (previously it depended on KMReaderWin). It is probably
possible, I just didn't know it is needed. I will work on it.
> I think the namespace should be the same as the name of the lib, e.g.
> MailViewer.
Ok, so we have MailViewer namespace. Just please suggest a name for the main
class (the viewer widget) as it is now called MailViewer. :)
> For krichtextwidget in kdelibs, the user passes an actioncollection, and
> krichtextwidget simply creates and adds the actions to there.
> krichtextwidget has means to control which actions are created.
The reason I don't like creating actions inside the lib is that you have to
expose them to the outside (as some users might not like the default
icon/shortcut/behavior of them). And as soon as you do, you loose control on
the KAction pointers.
Andras
_______________________________________________
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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic