<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="https://git.reviewboard.kde.org/r/117626/">https://git.reviewboard.kde.org/r/117626/</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;">I have to admit, for me this is a top development within Calligra this year in terms of importance and strategy!</pre>
 <br />









<p>- JarosÅ‚aw Staniek</p>


<br />
<p>On April 18th, 2014, 9:38 a.m. UTC, Inge Wallin wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Inge Wallin.</div>


<p style="color: grey;"><i>Updated April 18, 2014, 9:38 a.m.</i></p>









<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;">This patch implements the first simple version of a docx export filter. It has support for simple character formatting and paragraph formatting, including named styles. It can also distinguish between headers and normal paragraphs [This turned out not to be true when I look at the code in the RR now. But it will be before it's merged. We will concentrate on this now]. Other than that it's an unwritten page.  A special thanks to Lassi Nieminen who helped me with converting the styles.

The patch itself is very straightforward and mostly self contained in the filters/words/docx subdirectory. It builds on my previous work with libodfreader and libodf2, which are both in filters/. The only problem was that the KoZipStore and KoEncryptedStore backends create a file called "mimetype" automatically when a KoStore is created in write mode. I tried to work around this with as little impact as possible to the code in libs/odf and with full source compatibility with the previous API. If you think there is a better way to solve this problem than the one I implemented, then please tell me.</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;">Testing with a number of odt files. Lassi did all the testing involving MS Office since I don't have that.</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>CMakeLists.txt <span style="color: grey">(78421c5)</span></li>

 <li>filters/libodf2/KoOdfStyle.h <span style="color: grey">(558ade9)</span></li>

 <li>filters/libodf2/KoOdfStyle.cpp <span style="color: grey">(2b4eb95)</span></li>

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

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

 <li>filters/libodf2/KoOdfStyleProperties.h <span style="color: grey">(1bfbb5c)</span></li>

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

 <li>filters/libodfreader/OdtReader.h <span style="color: grey">(64e0584)</span></li>

 <li>filters/libodfreader/OdtReader.cpp <span style="color: grey">(6fa8ce6)</span></li>

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

 <li>filters/words/docx/export/README <span style="color: grey">(PRE-CREATION)</span></li>

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

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

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

 <li>libs/odf/KoDirectoryStore.h <span style="color: grey">(19c059d)</span></li>

 <li>libs/odf/KoDirectoryStore.cpp <span style="color: grey">(c893d47)</span></li>

 <li>libs/odf/KoEncryptedStore.h <span style="color: grey">(0edd892)</span></li>

 <li>libs/odf/KoEncryptedStore.cpp <span style="color: grey">(315df1a)</span></li>

 <li>libs/odf/KoStore.h <span style="color: grey">(dadecd1)</span></li>

 <li>libs/odf/KoStore.cpp <span style="color: grey">(fd42378)</span></li>

 <li>libs/odf/KoStore_p.h <span style="color: grey">(2e518c1)</span></li>

 <li>libs/odf/KoTarStore.h <span style="color: grey">(d99f09b)</span></li>

 <li>libs/odf/KoTarStore.cpp <span style="color: grey">(6829f34)</span></li>

 <li>libs/odf/KoZipStore.h <span style="color: grey">(90ffcb0)</span></li>

 <li>libs/odf/KoZipStore.cpp <span style="color: grey">(4235134)</span></li>

</ul>

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







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








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