<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/D25123">View Revision</a></tr></table><br /><div><div><p>A or C for me.</p>
<p><tt style="background: #ebebeb; font-size: 13px;">A</tt> looks more like how I imagine a "drop down terminal", but it's got a lot of margin below the bottom of it.</p>
<p>I tried seeing how it would look extended down to the bottom margin: <a href="https://phabricator.kde.org/F7728707" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7728707: Screenshot_20191103_135015.png</a></p>
<p><tt style="background: #ebebeb; font-size: 13px;">C</tt> looks more like a pull down projector screen, but that's not necessarily bad. It also doesn't quite reach the bottom.</p>
<p>I tried seeing how it would look extended down to the bottom margin: <a href="https://phabricator.kde.org/F7728733" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7728733: Screenshot_20191103_135310.png</a></p>
<p><a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">#vdg</a>, Should we allow some types of icons to be vertically off center? If not that, should we allow some icons to not reach all the way to the top and bottom margins? There are already some icons like that, but that doesn't necessarily mean we should want that.</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/D25123">https://phabricator.kde.org/D25123</a></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>