<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/102570/">http://git.reviewboard.kde.org/r/102570/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 11th, 2011, 4:35 a.m., <b>Thorsten Zachmann</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The fo:clip is a property of the style:graphic-property and can be used as that by all types of objects. Using it inside the pictureshape is there for wrong in my understanding as all shapes should support it.
Also there is a KoShape::setClipPath which makes it very simple to clip what can be painted by a shape. I think it would be good if we would use the same functionality for that patch. It is not as easy to do as odf according to fo:clip only supports rect but svg where this functionality comes from supports also individual path.
Using this will make it aslo very cheep to do the clipping.
So in my point of view the clip path should be loaded/saved in KoShape styles loading/saving. Maybe extend KoClipPath to also set a QRectF in it. Jan might be able to give some hints there.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I had a discussion with jahm on IRC yesterday and he too mentioned using KoShape::setClipPath. But KoImageData does caching of pixmaps which makes rendering faster. Also for a clipped image a smaller pixmap is loaded to memory in KoImageData against the whole shape loaded in KoShape.
Regarding your comment on clip path loading and saving we can discuss on IRC</pre>
<br />
<p>- Gopalakrishna</p>
<br />
<p>On September 10th, 2011, 5:26 p.m., Gopalakrishna Bhat 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.</div>
<div>By Gopalakrishna Bhat.</div>
<p style="color: grey;"><i>Updated Sept. 10, 2011, 5:26 p.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;">Adds support to fo:clip ODF property. In https://bugs.kde.org/show_bug.cgi?id=281531 the whole image is scaled and shown insted of clipping. With this patch pictureshape now supports clipping.
Known Issue: The doc file attacted to the bug report will be displayed as before because of the filter problem. If the MS doc file is converted to ODT using LO or OOo and then opened in Words the image is displayed similar to LO or OOo.</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;">Converted the MS doc file attached to https://bugs.kde.org/show_bug.cgi?id=281531 to ODT in LO and saw that the picture is displayed similarly in LO and Words.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=281531">281531</a>
</div>
<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/KoImageData.h <span style="color: grey">(3556100)</span></li>
<li>libs/flake/KoImageData.cpp <span style="color: grey">(da93926)</span></li>
<li>libs/flake/KoImageData_p.h <span style="color: grey">(6eba4bc)</span></li>
<li>plugins/pictureshape/PictureShape.h <span style="color: grey">(5c362fb)</span></li>
<li>plugins/pictureshape/PictureShape.cpp <span style="color: grey">(cba8cc4)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102570/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>