<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/D16633">View Revision</a></tr></table><br /><div><div><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>
<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></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>