<table><tr><td style="">alex-l added a comment.
</td></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/tosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@tosta</a></p>

<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#235383" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10243#235383</a>, <a href="https://phabricator.kde.org/p/tosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@tosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Gweview icon inspired by: <a href="https://commons.wikimedia.org/wiki/File:Circle-icons-eye.svg" class="remarkup-link" target="_blank" rel="noreferrer">Circle-icons-eye.svg</a></p>

<p><a href="https://phabricator.kde.org/F8685826" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8685826: gwenview.png</a></p>

<p>Update: submitted in Gwenview (<a href="https://phabricator.kde.org/T13400" class="remarkup-link" target="_blank" rel="noreferrer">T13400</a>)</p></div>
</blockquote>

<p>Here the gradient from blue to white in the center is definetely wrong... In Breeze icons you can see sligthly gradients but they mostly indicate a light source, they are not gradients from a color to another totally different one.</p>

<p>About the shape, it's cool but it's too much round compared to other shapes used in Breeze. And there are too much details namely all those circles.</p>

<p>Anyway this is just an eye, I think it's too generic as metaphor. In these cases you should provide reasoning on why this should be an improvement. Keep in mind that app icons, especially the core apps ones, shouldn't be updated without a strong reasoning.</p>

<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#235386" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10243#235386</a>, <a href="https://phabricator.kde.org/p/tosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@tosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>An icon proposal for Ark.</p>

<p>Update: Icon improvements and licensed LGPL.</p>

<p><a href="https://phabricator.kde.org/F8688969" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8688969: ark.png</a><br />
 <a href="https://phabricator.kde.org/F8688968" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8688968: ark.svg</a></p></div>
</blockquote>

<p>Here you totally missed the Breeze color palette... the gradient makes the icon look shiny and the reflection makes it look glossy. Don't you see we haven't anything like that in Breeze?</p>

<p>And the shape... why a circle should be better than the rectangle for an application to manage archives like Ark? I think the circle is hardly relatable to archives.</p>

<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#235432" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10243#235432</a>, <a href="https://phabricator.kde.org/p/tosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@tosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Colored Kwrite icon proposition:</p>

<p><a href="https://phabricator.kde.org/F8688852" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8688852: kwrite.png</a><br />
 <a href="https://phabricator.kde.org/F8688851" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8688851: kwrite.svg</a></p></div>
</blockquote>

<p>DON'T write labels in the icon. This is so bad visually as small sizes and for localization... what if one just use generic names? Plasma has an option for that, I don't remember if it's the default.</p>

<p>Again the color has nothing to do with other Breeze icons... and there are too many details in center while the left border is basically inherited from flat design...</p>

<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#235450" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10243#235450</a>, <a href="https://phabricator.kde.org/p/tosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@tosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Okular icon proposal.</p>

<p><a href="https://phabricator.kde.org/F8689327" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8689327: okular.png</a><br />
 <a href="https://phabricator.kde.org/F8689326" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8689326: okular.svg</a></p></div>
</blockquote>

<p>This is not bad, it could be a Breeze icon, but the combination of that red and that blue is very bad. You will have a hard time trying to mix blue and red, they don't fit together because of color theory reasons. In this cases do some search on how to design color palettes to learn how to mix different colors.</p>

<p>Anyway you should really try other tasks concerning icons and visual design. Breeze icons for core apps are too important.</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>tosta, johanneszarl, Leon0402, IohannesPetros, alex-l, starbuck, cullmann, IlyaBizyaev, arrowd, abetts, stikonas, knauss, 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, azyx, dmenig, manueljlin, Orage, cblack, konkinartem, ian, jguidon, Ghost6, jraleigh, fbampaloukas, squeakypancakes, alexde, GB_2, crozbo, firef, alexeymin, skadinna, genaxxx, aaronhoneycutt, jriddell, mbohlender<br /></div>