<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/102645/">http://git.reviewboard.kde.org/r/102645/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra.</div>
<div>By Thorsten Zachmann.</div>
<p style="color: grey;"><i>Updated Sept. 19, 2011, 5:16 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Use the size of the base class as proposed by Jan and use proper d-pointer . Additionally to the last patch invalidate the groups size when a shape is added/removed from the group. I will remove the kDebug statements before committing but left them there if people want to play with the patch to see if the size is updated properly.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">With the fix for the group size now every time the function is called it calculates the size from its children.
When testing this with cstester I found problmes in the documents I uploaded to.
http://www.zagge.de/anchor/groupsize/
The problem there is that due the very huge amount of shapes in the group it takes now quite a very very long time to render the pages.
The patch fixes that problem by adding a cache to the group size and only recalculates the size when there was a change.
It additionally don't forward the ChildChanged when a CollisionDetection is encountered. The CollisonChanged is not needed to forward to any parent as if the results in a change this change will be probagated.
The patch is a very fast way to see on how to fix the problem and for me to get feedback if the way is the right one to go forward.
I will definitely clean it up more before it is ready for inclusion.
The implementation of the shape container models needs to be moved to the cpp files and also the members will be put into the d pointer.
Looking forward to your feedback.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Before the patch switching between slides in the www.renaissance.co.il%2Fivbug%2Fmeeting87%2FArchitecture%20and%20design%20patterns.ppt file takes more then 10 seconds, with the patch it is instantly.
Tested resizing of groups of groups, resizing/modifying of shapes in groups. </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>libs/flake/KoShapeContainerModel.cpp <span style="color: grey">(7995236)</span></li>
<li>libs/flake/KoShapeGroup.h <span style="color: grey">(a53d2ad)</span></li>
<li>libs/flake/KoShapeGroup.cpp <span style="color: grey">(409b18c)</span></li>
<li>libs/flake/KoShape_p.h <span style="color: grey">(d055056)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102645/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>