<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/118643/">https://git.reviewboard.kde.org/r/118643/</a>
     </td>
    </tr>
   </table>
   <br />





<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for Calligra, Camilla Boemann, Lassi Nieminen, mojtaba shahi, and Thorsten Zachmann.</div>
<div>By Inge Wallin.</div>


<p style="color: grey;"><i>Updated Aug. 10, 2014, 4:02 p.m.</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Fixed all the issues.  As far as I see, this can be merged now.

It still doesn't contain support for tables, but moji is working on this slowly and the filter is already a little useful.</pre>
  </td>
 </tr>
</table>







<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch contains an export filter for wiki pages from odt files. It's done by request of the KDE documentation people, mainly mamarok.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Note that not all of the wiki spec is supported yet. Most notably, tables are not supported yet but that is the main priority for the next feature inclusion. Also links needs more work. But there is support for lists, headers, some general formatting and of course text in general.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">(Originally moji was going to put up this review but he got a pain in his hands so I offered to do it instead. I may have missed some important comment but I'm sure he will add to the comments then.)</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Lots of manual testing during development</p></pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>filters/libodf2/CMakeLists.txt <span style="color: grey">(8829a3c)</span></li>

 <li>filters/libodf2/KoOdfListLevelProperties.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/libodf2/KoOdfListLevelProperties.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/libodf2/KoOdfListStyle.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/libodf2/KoOdfListStyle.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/libodf2/KoOdfStyleManager.h <span style="color: grey">(9bc51f4)</span></li>

 <li>filters/libodf2/KoOdfStyleManager.cpp <span style="color: grey">(7e558c7)</span></li>

 <li>filters/libodf2/KoOdfStyleProperties.cpp <span style="color: grey">(f9d6d10)</span></li>

 <li>filters/libodfreader/OdfTextReader.h <span style="color: grey">(f5636e4)</span></li>

 <li>filters/libodfreader/OdfTextReader.cpp <span style="color: grey">(31dfba5)</span></li>

 <li>filters/libodfreader/OdfTextReaderBackend.h <span style="color: grey">(4c82a02)</span></li>

 <li>filters/libodfreader/OdfTextReaderBackend.cpp <span style="color: grey">(b6beb7d)</span></li>

 <li>filters/words/CMakeLists.txt <span style="color: grey">(805ca28)</span></li>

 <li>filters/words/wiki/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/OdfReaderWikiContext.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/OdfReaderWikiContext.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/OdtReaderWikiBackend.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/OdtReaderWikiBackend.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/WikiExport.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/WikiExport.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/export/words_wiki_export.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wiki/wiki-format.xml <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/118643/diff/" style="margin-left: 3em;">View Diff</a></p>






  </td>
 </tr>
</table>




  </div>
 </body>
</html>