From 95459985e015dd632e13de1963d1a1f36d1ff708 Mon Sep 17 00:00:00 2001 From: beefchimi Date: Wed, 16 Oct 2019 09:17:36 -0400 Subject: [PATCH 1/2] :bug: [SASS] Fix recolor-icon mixin --- src/styles/shared/_icons.scss | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/styles/shared/_icons.scss b/src/styles/shared/_icons.scss index 5281a2df065..a4c12166ca5 100644 --- a/src/styles/shared/_icons.scss +++ b/src/styles/shared/_icons.scss @@ -5,6 +5,10 @@ @mixin recolor-icon($fill-color, $secondary-color: null, $filter-color: null) { svg { fill: $fill-color; + + @if $secondary-color != null { + color: $secondary-color; + } } @if $filter-color != null { @@ -12,10 +16,6 @@ filter: $filter-color; } } - - @if $secondary-color != null { - color: $secondary-color; - } } @mixin color-icon($value, $hue: base) { From 7fd831abfd5b4fc5dd56b2d39726447e61e65898 Mon Sep 17 00:00:00 2001 From: beefchimi Date: Wed, 16 Oct 2019 09:22:44 -0400 Subject: [PATCH 2/2] :pencil: [CHANGELOG] Update docs --- UNRELEASED.md | 1 + 1 file changed, 1 insertion(+) diff --git a/UNRELEASED.md b/UNRELEASED.md index 3b3b42df453..b9fba8c1bfb 100644 --- a/UNRELEASED.md +++ b/UNRELEASED.md @@ -15,6 +15,7 @@ Use [the changelog guidelines](https://git.io/polaris-changelog-guidelines) to f ### Bug fixes - Doesn't render `MenuActions` if no actions are passed to an `actionGroups` item inside `Page` ([2266](https://github.com/Shopify/polaris-react/pull/2266))### Documentation +- Fixed `recolor-icon` Sass mixin to properly scope `$secondary-color` to the child `svg` ([#2298](https://github.com/Shopify/polaris-react/pull/2298)) ### Development workflow