D14116: Use a better icon for "Remove" on the widget handle

2018-07-15 Thread Nathaniel Graham
ngraham added a comment. Found it. Here's the patch: D14138: Use a trash icon for anything that will remove a widget or a panel REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D14116 To: ngraham, #plasma, #vdg,

D14116: Use a better icon for "Remove" on the widget handle

2018-07-15 Thread Nathaniel Graham
ngraham added a comment. Hmm actually that's for the toolbar above the panel, not the little menus that appear when you hover the mouse over a panel widget. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D14116 To: ngraham, #plasma, #vdg, davidedmundson Cc:

D14116: Use a better icon for "Remove" on the widget handle

2018-07-15 Thread Nathaniel Graham
ngraham added a comment. In D14116#292301 , @broulik wrote: > Can you make the same change also for the panel? There it's even more confusing as the "X" looks like it would close the popup but it's actually for deleting the widget. > Code is

D14116: Use a better icon for "Remove" on the widget handle

2018-07-15 Thread Kai Uwe Broulik
broulik added a comment. Can you make the same change also for the panel? There it's even more confusing as the "X" looks like it would close the popup but it's actually for deleting the widget. Code is in

D14116: Use a better icon for "Remove" on the widget handle

2018-07-14 Thread Nathaniel Graham
ngraham closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D14116 To: ngraham, #plasma, #vdg, davidedmundson Cc: gregormi, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14116: Use a better icon for "Remove" on the widget handle

2018-07-14 Thread Nathaniel Graham
ngraham added a comment. Ah yes, I've accidentally removed my panel widgets that way too! I'll submit a similar patch for that shortly. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D14116 To: ngraham, #plasma, #vdg Cc: gregormi, plasma-devel, ragreen,

D14116: Use a better icon for "Remove" on the widget handle

2018-07-14 Thread gregormi
gregormi added a comment. +1. I have a similar experience with the red X-Button as shown here for the Activity Manager: F6104707: grafik.png REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D14116 To: ngraham,

D14116: Use a better icon for "Remove" on the widget handle

2018-07-14 Thread Nathaniel Graham
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