D23942: Add enablefont and disablefont icon for kfontinst KCM
Noah Davis
noreply at phabricator.kde.org
Sat Sep 14 19:07:24 BST 2019
ndavis added a comment.
In D23942#531337 <https://phabricator.kde.org/D23942#531337>, @GB_2 wrote:
> In D23942#531330 <https://phabricator.kde.org/D23942#531330>, @ngraham wrote:
>
> > Shouldn't the checkmark be green?
>
>
> FWIW, both should be black/white. Disabling is not a destructive action like removing.
We use red for disabling as well. Perhaps we need to work on this a bit. One of the problems with only using black and white is that the emblems become harder to notice.
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D23942
To: guoyunhe, #breeze, ndavis
Cc: GB_2, ndavis, broulik, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190914/32ea0914/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list