<table><tr><td style="">ngraham accepted this revision.<br />ngraham 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/D18379">View Revision</a></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/D18379#396445" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18379#396445</a>, <a href="https://phabricator.kde.org/p/ndavis/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ndavis</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/D18379#396432" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18379#396432</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Very nice! Just what I had in mind. Do we really need a 32px version though?</p></div>
</blockquote>

<p>A number of icons start with <tt style="background: #ebebeb; font-size: 13px;">tools-</tt>, which means that this icon is the last fallback option. It could end up being seen in non-KDE apps, so I want to make sure I've covered my bases with the sizes.</p></div>
</blockquote>

<p>Ah OK, that makes a lot of sense. +1</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I thought we were trying to move towards all ≥ 32px icons being colorful, and not using the 32px size for action icons?</p></blockquote>

<p>I was not aware of us deciding not to use 32px for action icons. I'm not necessarily opposed to that idea, but it raises a number of questions about what to do with our own apps. For instance, should Okular start using color icons for its sidebar buttons when the icon size is set to Normal or Large and continue using monochrome icons for Small?</p></blockquote>

<p>Maybe it was just an informal conversation that I took too seriously. :) We can figure out later whether or not that's something we want to formally change (+1 on Okular using colorful icons for the large sizes though).</p>

<p>Ship it!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>BRANCH</strong><div><div>tools-icon (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18379">https://phabricator.kde.org/D18379</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, GB_2, ngraham<br /><strong>Cc: </strong>ngraham, GB_2, rooty, kde-frameworks-devel, michaelh, bruns<br /></div>