<table><tr><td style="">davidre 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/D27272">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/D27272#608602" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27272#608602</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I think the emblem icons look great in your "After" screenshot.</p>

<p>Wouldn't it be easier to have KDevelop use these emblem icons from their current names rather than symlinking the icons you want to use to new files with different names? Semantically it seems wrong; status icons are monochrome and use the action icon style. See <a href="https://hig.kde.org/style/icons/action_status.html" class="remarkup-link" target="_blank" rel="noreferrer">https://hig.kde.org/style/icons/action_status.html</a>.</p></div>
</blockquote>

<p>The reason for this patch is the same concern. Using the emblem icons here is also semantically wrong[1]. While it gives good results with Breeze and other themes like Oxygen or Adwaita don't have them and we can use the fallback of <tt style="background: #ebebeb; font-size: 13px;">dialog-*</tt> we don't know if it's true for every third party theme.<br />
[1] <a href="https://specifications.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html" class="remarkup-link" target="_blank" rel="noreferrer">https://specifications.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Emblems    Icons for tags and properties of files, that are displayed in the file manager. This context contains emblems for such things as “read-only” or “photos”.</p></blockquote></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/D27272">https://phabricator.kde.org/D27272</a></div></div><br /><div><strong>To: </strong>kossebau, VDG, ndavis<br /><strong>Cc: </strong>ngraham, davidre, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns<br /></div>