<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/105373/">http://git.reviewboard.kde.org/r/105373/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On July 7th, 2012, 10:04 a.m., <b>Boudewijn Rempt</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;">Maybe I did something wrong, but after building and installing everything, I still have badges in .kra and .ora files that I just created.</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;">Could be for two reasons (hopefully):
1. the old thumbnail kio-slave was still running with the old plugins loaded.
2. calligrathumbnail.desktop is still installed (in $(YOUR_INSTALL_BASE)/share/kde4/services/) which makes the thumbnail system pick up calligrathumbnail and not calligraimagethumbnail (because the thumbnail system currently works on a first-in-random-list-picked)

So please kill any kio_thumbnail process, rm installed calligrathumbnail.desktop, clean the thumbnail cache (~/.thumbnails/) and try again.</pre>
<br />








<p>- Friedrich W. H.</p>


<br />
<p>On July 4th, 2012, 1:08 p.m., Friedrich W. H. Kossebau 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 and Boudewijn Rempt.</div>
<div>By Friedrich W. H. Kossebau.</div>


<p style="color: grey;"><i>Updated July 4, 2012, 1:08 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;">Currently the Calligra thumbnailer is slightly broken:
* it claims to support a fixed set of mimetypes, even if not all of Calligra is built.
* it reports the flags like BlendIcon based on the last document a preview was done for
* it does not offer to create mimetypes for formats supported by import filters
* it might not really wait for the rendering of the loaded document

Attached patch fixes that by:
* duplicate into two plugins:
  calligraimagethumbnail for preview without blending the icon (for all image formats),
  calligrathumbnail for preview with blending the icon (for all other)
* wait for result of thumbnail rendering
* extends the range of supported mimetypes with the MS formats and some other otherwise not done.
* install separate thumbnail descriptions per import filter and only if
  the import filter is installed
* put a white background behind extracted mimetypes (as OO/LI creates transparent backgrounds which look bad then)

But this uncovers some problems:
First is that at least for xfig images the imagethumbnailer of kde-runtime hides the Calligra-based one. A patch to fix this is commited to kde-runtime for 4.8.5 and 4.9.0.
Second is that the result of some KoDocument subclasses is not really perfect, Words and Flow/Stage have seen fixes, but at least Sheets still needs one. Separate issue, separate patch, as this also is valid for the thumbnails added on storing ODS.

And some filters could possibly use a preview image from the original format? But that is also outside the scope of this patch.

Okay to backport to 2.5? Would ask for string freeze exception for all the separate mimetypes.</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/flow/visio/import/CMakeLists.txt <span style="color: grey">(0f42264)</span></li>

 <li>filters/flow/visio/import/flow_vsdx_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/karbon/karbon1.x/CMakeLists.txt <span style="color: grey">(127a2aa)</span></li>

 <li>filters/karbon/karbon1.x/karbon_karbon1x_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/karbon/xfig/CMakeLists.txt <span style="color: grey">(b3b21a9)</span></li>

 <li>filters/karbon/xfig/karbon_xfig_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/sheets/excel/import/CMakeLists.txt <span style="color: grey">(7ae3cbb)</span></li>

 <li>filters/sheets/excel/import/sheets_excel_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/sheets/xlsx/CMakeLists.txt <span style="color: grey">(a637afa)</span></li>

 <li>filters/sheets/xlsx/sheets_xlsx_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/stage/kpr2odf/CMakeLists.txt <span style="color: grey">(96b5650)</span></li>

 <li>filters/stage/kpr2odf/stage_kpr_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/stage/powerpoint/CMakeLists.txt <span style="color: grey">(8e5478c)</span></li>

 <li>filters/stage/powerpoint/stage_powerpoint_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/stage/pptx/CMakeLists.txt <span style="color: grey">(3332270)</span></li>

 <li>filters/stage/pptx/stage_pptx_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

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

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

 <li>filters/words/msword-odf/CMakeLists.txt <span style="color: grey">(2f79f7b)</span></li>

 <li>filters/words/msword-odf/words_msword_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/rtf/import/CMakeLists.txt <span style="color: grey">(4a6fe10)</span></li>

 <li>filters/words/rtf/import/words_rtf_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/wordperfect/import/CMakeLists.txt <span style="color: grey">(5efb0b0)</span></li>

 <li>filters/words/wordperfect/import/words_wpd_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>filters/words/works/import/CMakeLists.txt <span style="color: grey">(83f5254)</span></li>

 <li>filters/words/works/import/words_wps_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>krita/plugins/formats/ora/CMakeLists.txt <span style="color: grey">(666de52)</span></li>

 <li>krita/plugins/formats/ora/krita_ora_thumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tools/thumbnail/CMakeLists.txt <span style="color: grey">(f7aac3f)</span></li>

 <li>tools/thumbnail/calligracreator.h <span style="color: grey">(dfe50d2)</span></li>

 <li>tools/thumbnail/calligracreator.cpp <span style="color: grey">(804b0db)</span></li>

 <li>tools/thumbnail/calligraodgthumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tools/thumbnail/calligrathumbnail.desktop <span style="color: grey">(2c2272c)</span></li>

 <li>tools/thumbnail/kritathumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tools/thumbnail/otherofficethumbnail.desktop <span style="color: grey">(e62eba1)</span></li>

 <li>tools/thumbnail/sheetsodsthumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tools/thumbnail/stageodpthumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tools/thumbnail/wordsodtthumbnail.desktop <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

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




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








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