<table><tr><td style="">mart 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/D19392">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/D19392#421923" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19392#421923</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/D19392#421903" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19392#421903</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>Actually we're discussing in VDG-land whether or not this is something we should do anyway, because right now we have no way of forcing the use of a monochrome icon for a >22px size when a larger colorful version exists. Using <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt> to suffix the monochrome versions would allow us to do this. Adding some VDG folks for comment.</p></div>
</blockquote>

<p>The problem with using <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt> for all monochrome icons is that we'd have to create color icons without <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt> to replace the old monochrome icons without <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt>. That means we'd be effectively reworking a great deal of Breeze icons. Outside of a few instances in System Settings, Plasmashell and KWin titlebars, where we want to use color icons at small sizes, Breeze looks pretty consistent already, even with 3rd party applications. In places like the Digital Clock widget settings, where a monochrome icon is used when a color icon should be used, that can be fixed in breeze-icons just by making a new color icon at the appropriate size. If all of our monochrome icons had to have <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt>, then we'd have to depend on 3rd party applications using <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt> to keep a consistent look as well.</p>

<p>In short, we'd have to create a ton of new icons just to fix an issue that mainly exists in only a few pieces of our own software if we went the route with <tt style="background: #ebebeb; font-size: 13px;">-symbolic</tt>. Maybe with a future icon theme, we can plan things out better, but I don't think we should do anything that greatly changes the look of breeze-icons.</p></div>
</blockquote>

<p>exactly, whether is monochrome or not depends from the file and should be the underlying system be capable of recognizing which is which</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19392">https://phabricator.kde.org/D19392</a></div></div><br /><div><strong>To: </strong>mart, Kirigami<br /><strong>Cc: </strong>ngraham, GB_2, ndavis, VDG, cfeck, davidedmundson, plasma-devel, domson, dkardarakos, apol, mart, hein<br /></div>