ngraham created this revision. ngraham added reviewers: Plasma, VDG. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. ngraham requested review of this revision.
REVISION SUMMARY I often find myself clicking the Remove 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. TEST PLAN [image does here] REPOSITORY R119 Plasma Desktop BRANCH widget-delete-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D14116 AFFECTED FILES containments/desktop/package/contents/ui/AppletHandle.qml To: ngraham, #plasma, #vdg Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart