[gtk/gtk-4-2: 37/91] theme: Use -gtk-icon-filter for full-color icons

[Matthias Clasen <matthiasc src gnome org>]

commit eabbc0ef3069dc572e93672f2c0c19a59d07f951
Author: nana-4 <hnmaigo gmail com>
Date:   Sun May 31 06:37:06 2020 +0900

    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.

 gtk/theme/Adwaita/_common.scss | 20 ++++++--------------
 1 file changed, 6 insertions(+), 14 deletions(-)
---
diff --git a/gtk/theme/Adwaita/_common.scss b/gtk/theme/Adwaita/_common.scss
index ac5434c8cb..9d02478c1d 100644
--- a/gtk/theme/Adwaita/_common.scss
+++ b/gtk/theme/Adwaita/_common.scss
@@ -38,19 +38,9 @@ dnd {
   -gtk-icon-size: 32px;
 }
 
-/*
-   These wildcard seems unavoidable, need to investigate.
-   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); }
+image:disabled {
+  -gtk-icon-filter: opacity(0.5);
+}
 
 .view,
 %view {
@@ -508,7 +498,8 @@ button {
         @include button(backdrop);
 
         transition: $backdrop_transition;
-        -gtk-icon-filter: none;
+
+        &:not(:disabled) { -gtk-icon-filter: none; }
 
         &:active,
         &:checked { @include button(backdrop-active); }
@@ -524,6 +515,7 @@ button {
 
     &:disabled {
       @include button(insensitive);
+      -gtk-icon-filter: opacity(0.5);
 
       &:active,
       &:checked { @include button(insensitive-active); }