<table><tr><td style="">astippich 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/D10492">View Revision</a></tr></table><br /><div><div><p>Sorry for being so late to the party, but I just found this while reading the Frameworks 5.44 release notes.<br />
First of all, I am really grateful for the new icon. Actually, it would be really cool if we could also get a colorized version of a "media-artist" icon, since we are using a monochrome one there throughout Elisa :)</p>

<p>However, I think how the new icons are implemented here is problematic. You added icons with the same name as the monochrome ones, but the new ones look totally different. Depending on the icon size Elisa is requesting, we now get a different icon. This is also the reason why I created <a href="https://phabricator.kde.org/D10293" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D10293</a>.<br />
Elisa is also using some color overlay effects, which now totally brake. I upgraded to Frameworks 5.44, and now Elisa looks like this:</p>

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

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

<p>The icon for the tracks view looks now out of place, and doesn't work with the color overlay.</p>

<p>Again, thanks for the icons, but I think they should be implemented with different names, so that we must add explicit support for them in Elisa. This may also effect other applications similarly.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10492">https://phabricator.kde.org/D10492</a></div></div><br /><div><strong>To: </strong>andreask<br /><strong>Cc: </strong>astippich, ngraham, Frameworks, michaelh<br /></div>