<table><tr><td style="">kossebau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20166">View Revision</a></tr></table><br /><div><div><p>While the main purpose of this patch is to have SVG files in the repo, the processing to optimize runtime load is an interesting side-effect.</p>

<p>If you are curious about some numbers, here the install size estimated with <tt style="background: #ebebeb; font-size: 13px;">du -chb</tt> for the respective subfolders <tt style="background: #ebebeb; font-size: 13px;">desktoptheme/$theme</tt>, with <tt style="background: #ebebeb; font-size: 13px;">strip</tt> meaning the flags to svgcleaner as in the current patch:</p>

<div class="remarkup-table-wrap"><table style="border-collapse: separate;
          border-spacing: 1px;
          background: #d3d3d3;
          margin: 12px 0;">
<tr><td style="background: #ffffff;
          padding: 3px 6px;"></td><td style="background: #ffffff;
          padding: 3px 6px;">breeze</td><td style="background: #ffffff;
          padding: 3px 6px;">air</td><td style="background: #ffffff;
          padding: 3px 6px;">oxygen</td></tr>
<tr><td style="background: #ffffff;
          padding: 3px 6px;">old:</td><td style="background: #ffffff;
          padding: 3px 6px;">1.236.245</td><td style="background: #ffffff;
          padding: 3px 6px;">1.762.968</td><td style="background: #ffffff;
          padding: 3px 6px;">864.292</td></tr>
<tr><td style="background: #ffffff;
          padding: 3px 6px;">strip+z:</td><td style="background: #ffffff;
          padding: 3px 6px;">1.105.389</td><td style="background: #ffffff;
          padding: 3px 6px;">1.605.054</td><td style="background: #ffffff;
          padding: 3px 6px;">739.340</td></tr>
<tr><td style="background: #ffffff;
          padding: 3px 6px;">z:</td><td style="background: #ffffff;
          padding: 3px 6px;">1.231.078</td><td style="background: #ffffff;
          padding: 3px 6px;">1.762.734</td><td style="background: #ffffff;
          padding: 3px 6px;">875.083</td></tr>
<tr><td style="background: #ffffff;
          padding: 3px 6px;">strip:</td><td style="background: #ffffff;
          padding: 3px 6px;">2.250.847</td><td style="background: #ffffff;
          padding: 3px 6px;">4.071.777</td><td style="background: #ffffff;
          padding: 3px 6px;">3.164.541</td></tr>
<tr><td style="background: #ffffff;
          padding: 3px 6px;">svg 1:1:</td><td style="background: #ffffff;
          padding: 3px 6px;">3.130.982</td><td style="background: #ffffff;
          padding: 3px 6px;">5.624.588</td><td style="background: #ffffff;
          padding: 3px 6px;">4.777.657</td></tr>
<tr></tr>
</table></div>

<p>So with Oxygen there is a small increase when using gzip only, compared to before. Seems for some files inkscape or what stored those files before was more efficient compared to <tt style="background: #ebebeb; font-size: 13px;">gzip -9</tt>.<br />
IMHO the increase of 10791 is not that awful, after all in the long term we surely want to recommend the usage of the stripped variant. And for Breeze, after all the default, the plain gzip variant already gains 5167, so even without stripping the install size is in total not that much affected.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20166">https://phabricator.kde.org/D20166</a></div></div><br /><div><strong>To: </strong>kossebau, Plasma, VDG<br /><strong>Cc: </strong>fvogt, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>