<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/108925/">http://git.reviewboard.kde.org/r/108925/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Much better, ship it!
I still think that the fix for the crashes and the other fixes should be committed separately.</pre>
<br />
<p>- Inge</p>
<br />
<p>On February 13th, 2013, 7:33 p.m. UTC, mojtaba shahi wrote:</p>
<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.</div>
<div>By mojtaba shahi.</div>
<p style="color: grey;"><i>Updated Feb. 13, 2013, 7:33 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;">I tested all documents that frinring sent to me, and found some problems in mobi (algorithm and in converting odf to mobi html) and in extracting
images (external images) and in epub (bad handleing in tag bookmark).
Fixed all problems.</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 documents that sent to me, all documents exported ok and without any problem.</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>filters/words/epub/OdtHtmlConverter.cpp <span style="color: grey">(373ef7e)</span></li>
<li>filters/words/epub/exportepub2.cpp <span style="color: grey">(461cc9c)</span></li>
<li>filters/words/mobi/OdtMobiHtmlConverter.cpp <span style="color: grey">(de5ff3f)</span></li>
<li>filters/words/mobi/PalmDocCompression.cpp <span style="color: grey">(1eff462)</span></li>
<li>filters/words/mobi/exportmobi.cpp <span style="color: grey">(fee2509)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108925/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>