| Summary: | Wrong Icon to Remove User in User Manager Settings | ||
|---|---|---|---|
| Product: | [Plasma] Breeze | Reporter: | francesco.rigotti |
| Component: | Icons | Assignee: | visual-design |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | kainz.a, kde |
| Priority: | NOR | ||
| Version: | 5.8.4 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/breeze-icons/b0cf5f30406caa40a1a765e7a21c3ced0d403578 | Version Fixed In: | |
| Attachments: | Screenshot of the dialog | ||
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 (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. 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 -7 icons-dark/actions/16/list-remove.svg M +10 -3 icons-dark/actions/22/list-remove.svg M +3 -7 icons-dark/actions/24/list-remove.svg M +4 -7 icons/actions/16/list-remove.svg M +10 -3 icons/actions/22/list-remove.svg M +3 -7 icons/actions/24/list-remove.svg https://commits.kde.org/breeze-icons/b0cf5f30406caa40a1a765e7a21c3ced0d403578 |
Created attachment 106822 [details] Screenshot of the dialog I believe that the remove button should have a 'minus' icon (opposite of 'plus' for the add button) instead of a 'forbidden' icon