<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/102107/">http://git.reviewboard.kde.org/r/102107/</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 and Casper Boemann.</div>
<div>By Sebastian Sauer.</div>
<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;">The problem is that if a table-cell does not define a table-border then we are still drawing such a table border as visual helper for the user. Those visual helper borders can be clipped off in some cases (I was so far only able to reproduce the problem with one document at zoom-level 63%) cause we don't take the width of that visual helper border into account on clipping.
The attached patch solves that the easy way by just adjusting the painters clipping in the textshape to be sure we don't cutoff cosmetic pens which are exactly at the border of the shape ( x-pos=0, y-pos=0, width=outlineRect().width(), height=outlineRect.height() ) and are therefore partly (as in 1/2 of them) are drawn outside of the outlineRect. Since the cosmetic pen has always a size of 1 pixel butting the half of could mean that the pixel isn't drawn any longer (not sure there why it only happens with some zoom-levels while it works fine with others).
Now the question is if there is a better way to prevent to have that pixel cutoff. Maybe even down the road somewhere in the textlayout-library?
</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>plugins/textshape/TextShape.cpp <span style="color: grey">(6fddca0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102107/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>