Fix join() and append() $separator error messages

Closes #738
sass · Jul 3, 2019 · 5ff434d · 5ff434d
1 parent 47e0cfc
commit 5ff434d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -35,6 +35,9 @@
   passed a string that contains characters that aren't represented as a single
   byte in UTF-16.
 
+* Improve the error message for an unknown separator name passed to the `join()`
+  or `append()` functions.
+
 ## 1.22.2
 
 ### JavaScript API

diff --git a/lib/src/functions/list.dart b/lib/src/functions/list.dart
@@ -63,7 +63,7 @@ final _join = BuiltInCallable(
     separator = ListSeparator.comma;
   } else {
     throw SassScriptException(
-        '\$$separator: Must be "space", "comma", or "auto".');
+        '\$separator: Must be "space", "comma", or "auto".');
   }
 
   var bracketed = bracketedParam is SassString && bracketedParam.text == 'auto'
@@ -91,7 +91,7 @@ final _append =
     separator = ListSeparator.comma;
   } else {
     throw SassScriptException(
-        '\$$separator: Must be "space", "comma", or "auto".');
+        '\$separator: Must be "space", "comma", or "auto".');
   }
 
   var newList = [...list.asList, value];