Clamp $alpha in adjust-color() with HSL args

Closes #708
sass · Jul 3, 2019 · 2105a13 · 2105a13
1 parent 7bfba04
commit 2105a13
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -16,6 +16,9 @@
 
   [plain-CSS invert]: https://developer.mozilla.org/en-US/docs/Web/CSS/filter-function/invert
 
+* The `adjust-color()` function no longer throws an error when a large `$alpha`
+  value is combined with HSL adjustments.
+
 * The `alpha()` function now produces clearer error messages when the wrong
   number of arguments are passed.
 

diff --git a/lib/src/functions/color.dart b/lib/src/functions/color.dart
@@ -414,7 +414,7 @@ final _adjust = BuiltInCallable("adjust", r"$color, $kwargs...", (arguments) {
         hue: color.hue + (hue ?? 0),
         saturation: (color.saturation + (saturation ?? 0)).clamp(0, 100),
         lightness: (color.lightness + (lightness ?? 0)).clamp(0, 100),
-        alpha: color.alpha + (alpha ?? 0));
+        alpha: (color.alpha + (alpha ?? 0)).clamp(0, 1));
   } else if (alpha != null) {
     return color.changeAlpha((color.alpha + (alpha ?? 0)).clamp(0, 1));
   } else {