change icon colour

Author: blboyle

polaris-react/UNRELEASED.md

@@ -19,7 +19,6 @@ Use [the changelog guidelines](/documentation/Versioning%20and%20changelog.md) t
 - Increased token coverage by replacing hard coded `border-width` values with tokens, replaced sass var with css custom property, removed unused sass vars, and updated `Banner` Secondary action styles ([5389](https://github.com/Shopify/polaris/pull/5389))
 - Created `icon-attention` and `surface-attention` color tokens ([5389](https://github.com/Shopify/polaris/pull/5389))
 - Increased token coverage by removing unnecessary `transitions` and `animations` with hard coded duration values ([5405](https://github.com/Shopify/polaris/pull/5405/))
-- Added optional visual density updates and zebra striping to `DataTable` ([5365](https://github.com/Shopify/polaris/pull/5365/))
 - Added optional visual density updates and zebra striping to `DataTable` ([#5365](https://github.com/Shopify/polaris/pull/5365/))
 
 ### Bug fixes

polaris-react/src/components/DataTable/DataTable.scss

@@ -181,8 +181,9 @@ $breakpoint: 768px;
 
     .Icon {
       opacity: 1;
+
       svg {
-        fill: var(--p-text-subdued);
+        fill: var(--p-icon-disabled);
       }
     }
   }
@@ -192,8 +193,9 @@ $breakpoint: 768px;
 
     .Icon {
       opacity: 1;
+
       svg {
-        fill: var(--p-text-subdued);
+        fill: var(--p-icon-disabled);
       }
     }
   }
@@ -207,14 +209,11 @@ $breakpoint: 768px;
 .Cell-sorted {
   .Icon {
     opacity: 1;
-
   }
 
   .Heading:focus:not(:active) {
-    .Icon {
-      svg {
-        fill: var(--p-icon);
-      }
+    svg {
+      fill: var(--p-icon);
     }
   }
 }