<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Plasma, VDG.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />ngraham requested review of this revision.
</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/D14116">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>I often find myself clicking the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Remove</span></span></span> button and accidentally removing the widget itself when I'm just trying to hide the handle, because the current icon communicates "close this thing" better than "delete this thing". By switching it to a trash icon, we communicate its true meaning better.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>[image does here]</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>widget-delete-icon (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14116">https://phabricator.kde.org/D14116</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/AppletHandle.qml</div></div></div><br /><div><strong>To: </strong>ngraham, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>