<table><tr><td style="">mglb updated this revision to Diff 69507.<br />mglb 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/D25123">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/D25123#558363" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25123#558363</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);"><p>switching to that system means we would need to change many icons.</p></div>
</blockquote>

<p>Let's just keep this in mind when making new icons. At least when it doesn't stand out too much, i.e. we shouldn't make new square icons smaller than other square icons just to match circles, but also don't try to fill all available space when there is no need to.</p>

<p>I've aligned icons vertically. There are two new varians of "A":</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">A1 - Uses all available space, with larger top bar</li>
<li class="remarkup-list-item">A2 - Folder-icon sized, also with larger top bar. Dolphin/folder, system-run, and some other rectangular icons do not use full available area, even though they could just by scaling them up proportionally - they have 4px side margins and 6px up/down margins.</li>
</ul>

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

<p>A, A1:<br />
<a href="https://phabricator.kde.org/F7750323" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7750323: Screenshot_20191109_205001.png</a><a href="https://phabricator.kde.org/F7750324" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7750324: Screenshot_20191109_205010.png</a><br />
A2, C:<br />
<a href="https://phabricator.kde.org/F7750325" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7750325: Screenshot_20191109_205036.png</a><a href="https://phabricator.kde.org/F7750326" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7750326: Screenshot_20191109_205050.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D25123?vs=69202&id=69507">https://phabricator.kde.org/D25123?vs=69202&id=69507</a></div></div><br /><div><strong>BRANCH</strong><div><div>yakuake</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25123">https://phabricator.kde.org/D25123</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>icons-dark/apps/48/yakuake-a.svg<br />
icons-dark/apps/48/yakuake-a1.svg<br />
icons-dark/apps/48/yakuake-a2.svg<br />
icons-dark/apps/48/yakuake-b.svg<br />
icons-dark/apps/48/yakuake-c.svg<br />
icons-dark/apps/48/yakuake-d.svg<br />
icons-dark/apps/48/yakuake-e.svg<br />
icons-dark/apps/48/yakuake-f.svg<br />
icons-dark/apps/48/yakuake-g.svg<br />
icons/apps/48/yakuake-a.svg<br />
icons/apps/48/yakuake-a1.svg<br />
icons/apps/48/yakuake-a2.svg<br />
icons/apps/48/yakuake-b.svg<br />
icons/apps/48/yakuake-c.svg<br />
icons/apps/48/yakuake-d.svg<br />
icons/apps/48/yakuake-e.svg<br />
icons/apps/48/yakuake-f.svg<br />
icons/apps/48/yakuake-g.svg</div></div></div><br /><div><strong>To: </strong>mglb, VDG, Yakuake, hein<br /><strong>Cc: </strong>ndavis, KonqiDragon, ognarb, ngraham, hein, kde-frameworks-devel, vmarinescu, ryanmccoskrie, LeGast00n, GB_2, michaelh, bruns<br /></div>