<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 />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit a624f6349358a01275e0cb07d3b46bf107359f89 by Sebastian Sauer to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On July 28th, 2011, 11:32 a.m., Sebastian Sauer wrote:</p>
<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>
<p style="color: grey;"><i>Updated July 28, 2011, 11:32 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;">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>