Review Request: renaming style and add new style

[Pierre Stirnweiss]

And since this list uses the styleModel, I see the use case now, even if in
my mind it is a bit verging on the abuse of the model.
I am not so sure about the stylesModel API pollution (method
AddNewParagraphStyle) however. Is there a way to make clearer what this
really is? Such things otherwise tend to be wrongly used down the line
(undo/redo framework is a perfect example). I can already see someone in a
couple of months looking at the stylesModel API and see
AddNewParagraphStyle and think well, that is how you add a new style.
For example can't we make that a private method with a less generic name
and friend the appropriate classes?



On Wed, May 23, 2012 at 3:28 PM, C. Boemann <cbo at boemann.dk> wrote:

> On Wednesday 23 May 2012 15:25:26 Pierre Stirnweiss wrote:
> > But is this to be used only in the styleManager dialog then? What for,
> the
> > preview?
> Yes for the list in the style manager dialog
> _______________________________________________
> calligra-devel mailing list
> calligra-devel at kde.org
> https://mail.kde.org/mailman/listinfo/calligra-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120523/5f5916f7/attachment.htm>