<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/D21656">View Revision</a></tr></table><br /><div><div><p>I just tried the patch on my own computer and I noticed a few more things that need to be changed:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The canvas size is 128x128 px when it should be 48x48</li>
<li class="remarkup-list-item">Strokes need to be converted to paths ( <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Ctrl</kbd><span class="kbd-join" style="padding: 0 4px; color: #92969D;">+</span><kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Alt</kbd><span class="kbd-join" style="padding: 0 4px; color: #92969D;">+</span><kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">c</kbd> ). Strokes don't always look correct in various SVG renderers, including the Qt renderer.</li>
<li class="remarkup-list-item">I see you added a shadow inside the darker blue rim, but while you can do that, there also needs to be a 1px high shadow on the very bottom for it to match the HIG.<ul class="remarkup-list">
<li class="remarkup-list-item">There are some exceptions, but they are not common (or the icon creator and reviewers just forgot ¯\_(ツ)_/¯ ).</li>
</ul></li>
<li class="remarkup-list-item">The top left and bottom right corners are not aligned to the grid:</li>
</ul>

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

<p>Suggestions regarding the colors:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I think the icon would be more recognizable if the board had a darker color instead of a lighter color. The board itself could use a gradient, but I assume you're still planning to add that.</li>
<li class="remarkup-list-item">For the indentations, I think they would look more natural if their colors were based on the board's colors rather than the pieces' colors.</li>
</ul></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/D21656">https://phabricator.kde.org/D21656</a></div></div><br /><div><strong>To: </strong>ognarb, VDG<br /><strong>Cc: </strong>ngraham, ndavis, kde-frameworks-devel, LeGast00n, michaelh, bruns<br /></div>