Hello,<br><br>Found that 5 years ago Thomas Zander moved KoShapeGroup::boundingRect() to KoCommand, which now becomes KoShapeGroupCommandPrivate::containerBoundingRect(). I think now it's ok to move it back to KoShapeGroup, what do you think?<br>
<br><a href="https://projects.kde.org/projects/calligra/repository/revisions/fa024de913158d68bd5fe45fb734f28aaf8db74d">https://projects.kde.org/projects/calligra/repository/revisions/fa024de913158d68bd5fe45fb734f28aaf8db74d</a><br>
<br><div class="gmail_quote">On Tue, May 3, 2011 at 1:53 AM,  <span dir="ltr"><<a href="mailto:jaham@gmx.net">jaham@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On Thursday 28 April 2011 18:02:34 you wrote:<br>
> Also, I noticed that shadow.Insets is considered when calculating the<br>
> boundingRect to a shape, is it needed to include the shadow into the<br>
> boundingRect? And besides calculating boundingRect, where is shadow.Insets<br>
> being used?<br>
<br>
</div></div>Yes the shadow insets are there to expand the bounding rect with the shadows<br>
size so that e.g. canvas updates for that particular shape are calculated<br>
correctly.<br>
<br>
Ciao Jan<br>
</blockquote></div><br><br clear="all"><br>-- <br>Yue Liu<br>