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.<br>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.<br>
For example can't we make that a private method with a less generic name and friend the appropriate classes?<br><br><br><br><div class="gmail_quote">On Wed, May 23, 2012 at 3:28 PM, C. Boemann <span dir="ltr"><<a href="mailto:cbo@boemann.dk" target="_blank">cbo@boemann.dk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im">On Wednesday 23 May 2012 15:25:26 Pierre Stirnweiss wrote:<br>
> But is this to be used only in the styleManager dialog then? What for, the<br>
> preview?<br>
</div>Yes for the list in the style manager dialog<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
calligra-devel mailing list<br>
<a href="mailto:calligra-devel@kde.org">calligra-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/calligra-devel" target="_blank">https://mail.kde.org/mailman/listinfo/calligra-devel</a><br>
</div></div></blockquote></div><br>