davidhurka requested review of this revision.
davidhurka added a comment.
One potentially important thing: See inline comment.
INLINE COMMENTS
> filename-title-amarok.svg:8
> + }
> + .ColorScheme-ButtonFocus {
> + color:#3daee9;
I realized that http://notmart.org/blog/2016/05/icon-colors/ and
https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#Stylesheets
contratict in the class names. One uses `ColorScheme-Highlight`, the other
`ColorScheme-ButtonFocus`. breeze-icon-cleaner uses `ColorScheme-ButtonFocus`.
In the repository I see several icons which use `ColorScheme-Highlight`, and
only a few which use `ColorScheme-ButtonFocus`.
So which one is correct? I assume that
https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#Stylesheets
needs to be fixed.
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D28203
To: davidhurka, #vdg, ndavis, ngraham
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns
