<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/D22617">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/D22617#500346" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22617#500346</a>, <a href="https://phabricator.kde.org/p/davidhurka/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidhurka</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>By the way, the suggested stylesheet in <a href="https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#Breeze" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#Breeze</a> does not follow <a href="https://hig.kde.org/style/icon.html" class="remarkup-link" target="_blank" rel="noreferrer">https://hig.kde.org/style/icon.html</a> as far as I can understand it.</p></div>
</blockquote>

<p>I just changed the stylesheet in the workflow tips today. I need to update the HIG to match it.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Why does the stylesheet need a background?</li>
</ul></blockquote>

<p>You only need to add the classes that you will use. If you want to use the window background color, you can use the Background class.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">How are the color scheme properties in the stylesheet mapped to the reduced breeze palette in the HIG? (I think neutral or Beware Orange do not clearly map.)</li>
</ul></blockquote>

<p>Beware Orange is just the name of a color (it's an arbitrary name). NeutralText is a class that maps to the Neutral Text color in your system colorscheme. The color of Neutral Text just happens to be Beware Orange with the Breeze colorscheme, but you could make it purple by changing the colorscheme. Since Breeze uses Beware Orange, that color must be the default color in the stylesheet so that the icon color looks like Breeze in non-Qt apps.</p>

<p>If you find any more flaws in the HIG, report an issue at <a href="https://invent.kde.org/websites/hig-kde-org/issues" class="remarkup-link" target="_blank" rel="noreferrer">https://invent.kde.org/websites/hig-kde-org/issues</a> or fork, patch and merge request to submit a patch for that repo.</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/D22617">https://phabricator.kde.org/D22617</a></div></div><br /><div><strong>To: </strong>davidhurka, VDG, ndavis<br /><strong>Cc: </strong>Okular, kde-frameworks-devel, ngraham, LeGast00n, sbergeron, michaelh, bruns<br /></div>