<table><tr><td style="">filipf 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/D15999">View Revision</a></tr></table><br /><div><div><p>So what I suggested and tried to sketch in <a href="https://phabricator.kde.org/D16031" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16031</a> was to scale the icon down a bit inside the circles just so it's proportionally more satisfactory to the eye. I also removed the faint outer border to not oversaturate the new icons with elements. I may be wrong with my reasoning here of course, it's just my 2 cents.</p>

<p><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> I agree with you about the fuzzy edges from the shadows. Don't have too strong of an opinion about the translucency, although yeah maybe not as opaque as in the latest screenshots.</p>

<p>As for leaving the icon element transparent instead of the opposite color of the circle, scrap that suggestion because I've tested this with a theme I'm modifying for personal use and I don't believe this is a good idea because the wallpaper behind interferes with legibility:</p>

<p><a href="https://phabricator.kde.org/F6315927" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6315927: image.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15999">https://phabricator.kde.org/D15999</a></div></div><br /><div><strong>To: </strong>pstefan, ngraham, VDG<br /><strong>Cc: </strong>broulik, filipf, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>