[Breeze] [Bug 382650] Wrong Icon to Remove User in User Manager Settings

2018-02-02 Thread andreas
https://bugs.kde.org/show_bug.cgi?id=382650

andreas  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/bre
   ||eze-icons/b0cf5f30406caa40a
   ||1a765e7a21c3ced0d403578

--- Comment #3 from andreas  ---
Git commit b0cf5f30406caa40a1a765e7a21c3ced0d403578 by andreas kainz.
Committed on 02/02/2018 at 12:55.
Pushed by andreask into branch 'master'.

Update list-remove icon with - instead cancel icon

M  +4-7icons-dark/actions/16/list-remove.svg
M  +10   -3icons-dark/actions/22/list-remove.svg
M  +3-7icons-dark/actions/24/list-remove.svg
M  +4-7icons/actions/16/list-remove.svg
M  +10   -3icons/actions/22/list-remove.svg
M  +3-7icons/actions/24/list-remove.svg

https://commits.kde.org/breeze-icons/b0cf5f30406caa40a1a765e7a21c3ced0d403578

-- 
You are receiving this mail because:
You are watching all bug changes.

[Breeze] [Bug 382650] Wrong Icon to Remove User in User Manager Settings

2017-08-02 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=382650

--- Comment #2 from francesco.rigo...@gmail.com ---
(In reply to David Edmundson from comment #1)
> We use the icon list-remove, which is the opposite of list-add.
> 
> From a code POV it's right, I'll forward this to the icon people to have an
> opinion on whether the icon is correct

Thank you for checking this out.

Indeed the code is right, but I find the icon misleading.

list-remove-user is very clear, but then the add button should be updated to
the counterpart list-add-user.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Breeze] [Bug 382650] Wrong Icon to Remove User in User Manager Settings

2017-08-01 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=382650

Christoph Feck  changed:

   What|Removed |Added

   Keywords|junior-jobs |

-- 
You are receiving this mail because:
You are watching all bug changes.

[Breeze] [Bug 382650] Wrong Icon to Remove User in User Manager Settings

2017-07-28 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=382650

David Edmundson  changed:

   What|Removed |Added

  Component|kcontrol module |Icons
Product|user-manager|Breeze
   Assignee|j...@jriddell.org |visual-des...@kde.org
 CC||kain...@gmail.com,
   ||k...@davidedmundson.co.uk

--- Comment #1 from David Edmundson  ---
We use the icon list-remove, which is the opposite of list-add.

>From a code POV it's right, I'll forward this to the icon people to have an
opinion on whether the icon is correct

-- 
You are receiving this mail because:
You are watching all bug changes.