<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/D16421">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/D16421#349083" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16421#349083</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);"><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/D16421#349082" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16421#349082</a>, <a href="https://phabricator.kde.org/p/ndavis/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ndavis</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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/D16421#349081" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16421#349081</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>This would all require some additional changes in Dolphin and Folder view, of course.</p></div>
</blockquote>

<p>So should I change <tt style="background: #ebebeb; font-size: 13px;">emblem-added</tt> and <tt style="background: #ebebeb; font-size: 13px;">emblem-remove</tt> back to their original colors? It wouldn't be any worse than it currently is for Dolphin before any changes are applied to it.</p></div>
</blockquote>

<p>Well, we should discuss that first. :)</p>

<p><a href="https://lxr.kde.org/search?_filestring=&_string=emblem-added" class="remarkup-link" target="_blank" rel="noreferrer">An LXR search</a> reveals that all users of these emblem use them in the same way. We have two options for how we want to change things:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Use new iconography (such as my proposed checkmark) for <tt style="background: #ebebeb; font-size: 13px;">emblem-added</tt> and <tt style="background: #ebebeb; font-size: 13px;">emblem-remove</tt></li>
<li class="remarkup-list-item">Create new emblems for this and then over time change our apps to use them. This seems less desirable since then <tt style="background: #ebebeb; font-size: 13px;">emblem-added</tt> and <tt style="background: #ebebeb; font-size: 13px;">emblem-remove</tt> will then just be unused, and also it requires code changes to those apps.
<br /><br />
So if we want to further improve these emblems, my vote is for changing the existing icons rather than making new ones.</li>
</ul></div>
</blockquote>

<p>So keep the blue color for now?</p>

<p>Something that I plan to do in the future is re-do the package management icons and I was hoping to reuse emblem-added and emblem-removed for that. Currently, the package management icons are action icons, but they are hard to read and they blur the line between action and state the way they are used, which is why I'm planning to make them emblems instead.</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/D16421">https://phabricator.kde.org/D16421</a></div></div><br /><div><strong>To: </strong>ndavis, VDG<br /><strong>Cc: </strong>bruns, ngraham, bcooksley, kde-frameworks-devel, VDG, michaelh<br /></div>