<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 />





 <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 673631d68052c494f4cee7c9fcaed13f973219b3 by Mojtaba Shahi Senobari to branch calligra/2.6.</pre>
 <br />









<p>- Commit</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>