theme: Use -gtk-icon-filter for full-color icons

Not for symbolic icons.

Don't apply `-gtk-icon-filter: opacity(0.5);` to the symbolic icons as
they already have the "gray" colors indicating the disabled state.
Symbolic icons can be styled using the `color` property.

Also remove the obsolete comment.
This commit is contained in:
nana-4 2020-05-31 06:37:06 +09:00 committed by Matthias Clasen
parent bfcd5f4881
commit eabbc0ef30

View File

@ -38,19 +38,9 @@ dnd {
-gtk-icon-size: 32px; -gtk-icon-size: 32px;
} }
/* image:disabled {
These wildcard seems unavoidable, need to investigate. -gtk-icon-filter: opacity(0.5);
Wildcards are bad and troublesome, use them with care, }
or better, just don't.
Every time a wildcard is used a kitten dies, painfully.
*/
spinner:disabled,
arrow:disabled,
scrollbar:disabled,
check:disabled,
radio:disabled,
treeview.expander:disabled { -gtk-icon-filter: opacity(0.5); }
.view, .view,
%view { %view {
@ -508,7 +498,8 @@ button {
@include button(backdrop); @include button(backdrop);
transition: $backdrop_transition; transition: $backdrop_transition;
-gtk-icon-filter: none;
&:not(:disabled) { -gtk-icon-filter: none; }
&:active, &:active,
&:checked { @include button(backdrop-active); } &:checked { @include button(backdrop-active); }
@ -524,6 +515,7 @@ button {
&:disabled { &:disabled {
@include button(insensitive); @include button(insensitive);
-gtk-icon-filter: opacity(0.5);
&:active, &:active,
&:checked { @include button(insensitive-active); } &:checked { @include button(insensitive-active); }