<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/111044/">http://git.reviewboard.kde.org/r/111044/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
<tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra, C. Boemann, Boudewijn Rempt, and Thorsten Zachmann.</div>
<div>By Inge Wallin.</div>
<p style="color: grey;"><i>Updated Aug. 2, 2013, 6:39 a.m.</i></p>
<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;">This patch implements loading, saving and painting of borders for shapes in general. It also implements calls to actually paint borders in particular for the picture shape and text shape.
At the same time I moved border painting from KWCanvasBase to KoBorder, which I announced some weeks ago that I wanted to do.
A patch this deep in the libraries should not go in without a run of cstester. I would be glad for help with this since I only have a semi-weak laptop.</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;">Tested with all the files with the string "border" in it in the test suite.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>libs/flake/KoShape.h <span style="color: grey">(fe6cc3f)</span></li>
<li>libs/flake/KoShape.cpp <span style="color: grey">(6d14a8c)</span></li>
<li>libs/flake/KoShape_p.h <span style="color: grey">(aa6633c)</span></li>
<li>libs/odf/KoBorder.h <span style="color: grey">(bdbf86e)</span></li>
<li>libs/odf/KoBorder.cpp <span style="color: grey">(58f8e45)</span></li>
<li>libs/textlayout/FloatingAnchorStrategy.h <span style="color: grey">(eb0ff6b)</span></li>
<li>libs/textlayout/FloatingAnchorStrategy.cpp <span style="color: grey">(465f149)</span></li>
<li>libs/textlayout/KoTextDocumentLayout.cpp <span style="color: grey">(70f4ccd)</span></li>
<li>libs/textlayout/KoTextLayoutRootAreaProvider.h <span style="color: grey">(d8c7ac3)</span></li>
<li>libs/textlayout/KoTextShapeData.cpp <span style="color: grey">(1b424b7)</span></li>
<li>plugins/pictureshape/PictureShape.cpp <span style="color: grey">(976fadd)</span></li>
<li>plugins/textshape/SimpleRootAreaProvider.h <span style="color: grey">(62534a6)</span></li>
<li>plugins/textshape/SimpleRootAreaProvider.cpp <span style="color: grey">(63f1898)</span></li>
<li>plugins/textshape/TextShape.cpp <span style="color: grey">(b230f0a)</span></li>
<li>words/part/KWCanvasBase.cpp <span style="color: grey">(7d12890)</span></li>
<li>words/part/KWRootAreaProvider.h <span style="color: grey">(45b4870)</span></li>
<li>words/part/KWRootAreaProvider.cpp <span style="color: grey">(fc8660d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111044/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>