<table><tr><td style="">ndavis 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/D19324">View Revision</a></tr></table><br /><div><div><p>Much better. Now for some questions and suggestions.</p>

<p>Why does the icon use green on the top of the logo? The light theme version doesn't look bad, but isn't the official logo all blue? In general, we try to preserve the original branding.</p>

<p>Why is the dark theme version greener than the light theme? Normally we try to make color icons the same for light and dark themes.</p>

<p>It would be good to turn those shadows into gradients with <tt style="background: #ebebeb; font-size: 13px;">00000000</tt> as the color for the bottom right of the shadow. The darker shadow should expand to cover the whole area to the bottom left of the right side of the logo. Here's what I mean: <br />
<a href="https://phabricator.kde.org/F6637009" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6637009: Screenshot_20190226_052530.png</a><br />
<a href="https://phabricator.kde.org/F6637011" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6637011: Screenshot_20190226_052559.png</a></p>

<p>The parts of the logo in this icon are layered differently from the official logo: <a href="https://user-images.githubusercontent.com/49339/32078472-5053adea-baa7-11e7-9034-519002f12ac7.png" class="remarkup-link" target="_blank" rel="noreferrer">https://user-images.githubusercontent.com/49339/32078472-5053adea-baa7-11e7-9034-519002f12ac7.png</a></p>

<p>The right edge of the right side of the logo is a bit too dark relative to the inner edge.</p>

<p>If you're having trouble finding the right blues to make the logo look 3D, try using the shadows to separate the layers like I did in the screenshots above.</p>

<p>I'll have more to say later.</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/D19324">https://phabricator.kde.org/D19324</a></div></div><br /><div><strong>To: </strong>axionl, VDG<br /><strong>Cc: </strong>ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>