<table><tr><td style="">acrouthamel 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><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 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></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>