<table><tr><td style="">alex-l added a comment.
</td></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/T10243#202038" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10243#202038</a>, <a href="https://phabricator.kde.org/p/mglb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mglb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/T10243#201329" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10243#201329</a>, <a href="https://phabricator.kde.org/p/alex-l/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@alex-l</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>EDIT: I have already said this here and there but please don't run any automated script to SVG icons, because when opening them back in Inkscape they are corrupted and everytime I edit a Breeze icon I have to do additional work to fix shapes and gradients corrupted by the scripts.</p></div>
</blockquote>

<p>Samples please, it would be nice to fix inkscape and/or scripts.<br />
 Also, we could store original/inkscape-friendly svg files in repo and run optimization script as part of install process.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The diagonal thing in Konsole icon is to add some identity, without it we would have a classic terminal icon.</p></blockquote>

<p>Why "different"/non-classic icon is better than classic terminal icon? Classic one is already known and everyone will be able to find it in their menu.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Any ideas on how to add identity in general?</p></blockquote>

<p>Basic/core applications (i.e. KWrite, Konsole, Dolphin, etc) come with the system "by default" and users need to find them easily without knowing how an app icon looks like in specific desktop environment. I guess people see them as "built-in" functions, not as stand-alone applications.</p></div>
</blockquote>

<p>As I said in another comment an example of problem in Inkscape is ">" symbol in Konsole 48x48px icon. Btw those scripts are just to save package's size, there is no point to run them after installation, also because SVG icons are rendered once and then stored in the cache, so there is any advantage in making them optimized but reducing loading time for the first time the icon is shown.</p>

<p>If KDE wants their apps to have an identity, and this is what someone expressed in this thread, we can't ship just generic icon. In fact I see Konsole used without Plasma in /r/unixporn posts on Reddit.</p></div></div><br /><div><strong>TASK DETAIL</strong><div><a href="https://phabricator.kde.org/T10243">https://phabricator.kde.org/T10243</a></div></div><br /><div><strong>To: </strong>alex-l<br /><strong>Cc: </strong>Leon0402, IohannesPetros, alex-l, starbuck, cullmann, IlyaBizyaev, arrowd, abetts, stikonas, knauss, mglb, filipf, mludwig, aacid, lesliezhai, elvisangelaccio, kossebau, trickyricky26, ndavis, yurchor, KDE Games, Ark, KDE PIM, Discover Software Store, Yakuake, Kate, Okular, Gwenview, Konsole, KDE Applications, VDG, ngraham, cblack, konkinartem, ian, jguidon, hannahk, Ghost6, jraleigh, MrPepe, fbampaloukas, squeakypancakes, alexde, GB_2, crozbo, firef, alexeymin, skadinna, genaxxx, aaronhoneycutt, jriddell, mbohlender<br /></div>