From 17fd2c9f84be84fb917f40212d2ddd88fe6a6d1e Mon Sep 17 00:00:00 2001 From: schwastek <31923722+schwastek@users.noreply.github.com> Date: Tue, 26 Sep 2017 14:05:59 +0200 Subject: [PATCH] Improve theme-color-level() using abs() (#24074) `abs()` gets the absolute value of `$level`. That way, `if/else` statement can be completely eliminated. --- scss/_functions.scss | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/scss/_functions.scss b/scss/_functions.scss index a63680aa5c..5d43b57087 100644 --- a/scss/_functions.scss +++ b/scss/_functions.scss @@ -80,11 +80,7 @@ @function theme-color-level($color-name: "primary", $level: 0) { $color: theme-color($color-name); $color-base: if($level > 0, #000, #fff); + $level: abs($level); - @if $level < 0 { - // Lighter values need a quick double negative for the Sass math to work - @return mix($color-base, $color, $level * -1 * $theme-color-interval); - } @else { - @return mix($color-base, $color, $level * $theme-color-interval); - } + @return mix($color-base, $color, $level * $theme-color-interval); } -- 2.47.2