<table><tr><td style="">ngraham 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/D6313">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/D6313#238595" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D6313#238595</a>, <a href="https://phabricator.kde.org/p/acrouthamel/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@acrouthamel</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I'm just saying if you prefer colored over symbolic line art, that is something for <a href="https://phabricator.kde.org/tag/breeze/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#Breeze</a> or <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> to work out. Not that they are really taking advantage of anything.</p>

<p>The bug here is that the system isn't aware of scaling at the moment. So the icons it <strong>should</strong> be displaying are the symbolic ones, regardless of whether you are running HiDPI or not (and whether you like them or not). That's because the menu should be displaying 24x24 or 32x32 icons, or whatever it should be. But in HiDPI environments it is doubling that (or whatever scale you have set), and displaying icons from high-res folders such as 64x64 or 96x96, where they shouldn't be displayed.</p>

<p>This patch fixes that issue and forces the system to load the correct size, as originally intended. Menus and stuff shouldn't be loading icons from these high-res folders. Symbolic vs. colored is a design thing to work out separately.</p></div>
</blockquote>

<p>Right, and I'm saying we need to work that out before we fix this. Visually, the whole point of making line-art versions for full-color icons is to look better at really small sizes on low-resolution screens. If we fix this bug without resolving the issues in Breeze, then people with HiDPI screens will see line-art icons where the full-color ones would work fine and look better.</p>

<p>...Unless I'm misunderstanding something obvious, in which case, feel free to ignore my deluded ramblings!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R302 KIconThemes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6313">https://phabricator.kde.org/D6313</a></div></div><br /><div><strong>To: </strong>broulik, kde-frameworks-devel, Plasma, VDG<br /><strong>Cc: </strong>hein, rkflx, acrouthamel, ngraham, elvisangelaccio, mart, kvermette, cfeck, davidedmundson, plasma-devel, Frameworks, ragreen, Pitel, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>