<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/D16633">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/D16633#353308" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16633#353308</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>Thank you! Two comments:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I'm not sure using the negative color is appropriate here. That color is used for dangerous operations or error conditions, neither of which apply to the OS volume. It's true that you could potentially blow yourself up by messing with it, but I don't really think that fits. The current red icon for root was never appropriate in the first place IMHO. Either Icon Blue or just the regular Icon Grey color seem more appropriate to me.</li>
</ul></div>
</blockquote>

<p>I understand what you're saying, but throughout the Breeze theme, red is also used for root. If we used grey it would probably still be fine, but blue would stick out.</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">Could we experiment with using the Plasma logo instead of a slash? The idea of using a slash to represent the root disk will be immediately apparent to technical users, but will be lost on everyone else. The Plasma logo might be more easily comprehensible for everyone.</li>
</ul></blockquote>

<p>I do agree that the forward slash isn't a great symbol for root for non-technical users, but it's very difficult to fit anything even slightly complex onto the 16 px icons without it becoming difficult to read. The Plasma logo works fine for 64px, but it's not very good at the smaller sizes.</p>

<p>This is my best attempt at getting the plasma logo to fit on the 16px icon:<br />
<a href="https://phabricator.kde.org/F6388344" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6388344: Screenshot_20181103_120444.png</a></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/D16633">https://phabricator.kde.org/D16633</a></div></div><br /><div><strong>To: </strong>ndavis, VDG<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>