<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/D16905">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/D16905#361148" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16905#361148</a>, <a href="https://phabricator.kde.org/p/trickyricky26/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@trickyricky26</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Ok I will change that, however the graphic on the Icons HIG Page (<a href="https://hig.kde.org/_images/Breeze-icon-design-5.png" class="remarkup-link" target="_blank" rel="noreferrer">https://hig.kde.org/_images/Breeze-icon-design-5.png</a>) says a inner size of 28px which means 2px margins on each size. <br />
 So that graphic should be updated, or if the 4px margins only apply to action icons, that should be clearly stated in the text.<br />
 I don't recall either of them being adressed in <a href="https://phabricator.kde.org/D16848" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16848: Revamp Icon Design and Emblem pages</a>.</p></div>
</blockquote>

<p>Yes, our HIG still needs more work. My advice is that when you notice a broad inconsistency between what the HIG says and what actually exists, look at icons similar to what you're making and copy what you can from those. The exception here is with the new icon colors because they will not have a noticable difference in existing KDE apps where the color is changed by the colorscheme anyway.</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/D16905">https://phabricator.kde.org/D16905</a></div></div><br /><div><strong>To: </strong>trickyricky26, VDG, Breeze, ndavis<br /><strong>Cc: </strong>cfeck, emateli, ndavis, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>