<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/107753/">http://git.reviewboard.kde.org/r/107753/</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;">The patch itself looks fine.

But I wonder: Is it really necessary to install icons with that high resolution? I thought that's why we use SVG to begin with, to get scalability. In very low resolutions there is often need for manual tweaking but is this also true at 256x256 and up?</pre>
 <br />







<p>- Inge</p>


<br />
<p>On December 16th, 2012, 7:49 a.m., Yue Liu 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.</div>
<div>By Yue Liu.</div>


<p style="color: grey;"><i>Updated Dec. 16, 2012, 7:49 a.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;">Added retina compatible (1024x1024) icons for most apps.</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;">works on mac</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>braindump/data/icons/hi1024-app-braindump.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>braindump/data/icons/hi128-app-braindump.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>braindump/data/icons/hi256-app-braindump.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>braindump/data/icons/hi512-app-braindump.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>braindump/data/icons/hi64-app-braindump.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>braindump/src/CMakeLists.txt <span style="color: grey">(c009e20)</span></li>

 <li>flow/CMakeLists.txt <span style="color: grey">(571e87e)</span></li>

 <li>flow/part/CMakeLists.txt <span style="color: grey">(41a9ecf)</span></li>

 <li>flow/pics/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi1024-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi128-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi16-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi256-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi32-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi512-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hi64-app-calligraflow.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>flow/pics/hisc-app-calligraflow.svgz <span style="color: grey">(PRE-CREATION)</span></li>

 <li>karbon/pics/app/hi1024-app-karbon.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>karbon/pics/app/hi128-app-karbon.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>karbon/pics/app/hi16-app-karbon.png <span style="color: grey">(47f1a67)</span></li>

 <li>karbon/pics/app/hi22-app-karbon.png <span style="color: grey">(5ed414d)</span></li>

 <li>karbon/pics/app/hi256-app-karbon.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>karbon/pics/app/hi32-app-karbon.png <span style="color: grey">(9783605)</span></li>

 <li>karbon/pics/app/hi48-app-karbon.png <span style="color: grey">(09fb40f)</span></li>

 <li>karbon/pics/app/hi512-app-karbon.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>karbon/pics/app/hi64-app-karbon.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>karbon/pics/app/hisc-app-karbon.svgz <span style="color: grey">(e757469)</span></li>

 <li>kexi/CMakeLists.txt <span style="color: grey">(4afc425)</span></li>

 <li>krita/CMakeLists.txt <span style="color: grey">(6641610)</span></li>

 <li>krita/pics/hi1024-app-krita.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>krita/pics/hi512-app-krita.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>plan/pics/hi1024-app-calligraplan.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>plan/pics/hi256-app-calligraplan.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>plan/pics/hi512-app-calligraplan.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/CMakeLists.txt <span style="color: grey">(1544b58)</span></li>

 <li>sheets/data/pics/hi1024-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi128-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi16-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi16-app-sheets.png <span style="color: grey">(2fc3d86)</span></li>

 <li>sheets/data/pics/hi22-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi22-app-sheets.png <span style="color: grey">(9f8c807)</span></li>

 <li>sheets/data/pics/hi256-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi32-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi32-app-sheets.png <span style="color: grey">(5161357)</span></li>

 <li>sheets/data/pics/hi48-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi48-app-sheets.png <span style="color: grey">(b908ae9)</span></li>

 <li>sheets/data/pics/hi512-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hi64-app-calligrasheets.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sheets/data/pics/hisc-app-calligrasheets.svgz <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/part/CMakeLists.txt <span style="color: grey">(ab45f1d)</span></li>

 <li>stage/pics/hi1024-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi128-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi16-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi16-app-stage.png <span style="color: grey">(76bd391)</span></li>

 <li>stage/pics/hi22-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi22-app-stage.png <span style="color: grey">(d2a21d9)</span></li>

 <li>stage/pics/hi256-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi32-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi32-app-stage.png <span style="color: grey">(e75e9e4)</span></li>

 <li>stage/pics/hi48-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi48-app-stage.png <span style="color: grey">(0fff633)</span></li>

 <li>stage/pics/hi512-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hi64-app-calligrastage.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>stage/pics/hisc-app-calligrastage.svgz <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/part/CMakeLists.txt <span style="color: grey">(3dca430)</span></li>

 <li>words/pics/hi1024-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi128-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi16-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi16-app-words.png <span style="color: grey">(8c63f72)</span></li>

 <li>words/pics/hi22-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi22-app-words.png <span style="color: grey">(ef8fbef)</span></li>

 <li>words/pics/hi256-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi32-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi32-app-words.png <span style="color: grey">(60dc06d)</span></li>

 <li>words/pics/hi48-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi48-app-words.png <span style="color: grey">(5a401ec)</span></li>

 <li>words/pics/hi512-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi64-app-calligrawords.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hi64-app-words.png <span style="color: grey">(f6d448c)</span></li>

 <li>words/pics/hisc-app-calligrawords.svgz <span style="color: grey">(PRE-CREATION)</span></li>

 <li>words/pics/hisc-app-words.svgz <span style="color: grey">(3290137)</span></li>

</ul>

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




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








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