<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D7520" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I support this change, for the additional reason that it's appropriate for the "move to trash" icon to be black, because red icons are for destructive changes, and moving something to the trash isn't a destructive change (it's the prelude to one). The actually destructive change now gets the red icon.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R265 KConfigWidgets</div></div></div><br /><div><strong>BRANCH</strong><div><div>user-trash</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7520" rel="noreferrer">https://phabricator.kde.org/D7520</a></div></div><br /><div><strong>To: </strong>elvisangelaccio, Frameworks, ngraham<br /><strong>Cc: </strong>ngraham, andreaska, broulik<br /></div>