gerillass 1.5.0 → 1.6.1

package/scss/library/_text-selection.scss

@@ -14,5 +14,9 @@
     *::selection {
       @content;
     }
+  } @else if not & and $value == "only" {
+    @error "`only` has no meaning at the root of a stylesheet, because there is no selector to limit the effect to. Call `text-selection(\"only\")` inside a selector, or pass no argument at all.";
+  } @else {
+    @error "`#{$value}` is not a valid argument for `text-selection`. Pass `\"only\"` inside a selector to affect just that element, or pass no argument at all.";
   }
 }

package/scss/library/_text-shadow.scss

@@ -7,6 +7,13 @@
     // Takes each items come from $params.
     $item: nth($params, $i);
 
+    // Each group must carry at least a direction, a colour and a size. Without
+    // this the nth() calls below fail with "Invalid index", which says nothing
+    // about what the mixin wanted.
+    @if length($item) < 3 {
+      @error "`#{$item}` is not a valid shadow for `text-shadow`. Each group needs a direction, a colour and a size, such as `bottom-right red 5px`. An optional blur and a fill flag may follow.";
+    }
+
     // The first value comes from the $item is direction.
     // This will be used in the @if, @else statements with $size to control the offset of the shadows.
     $direction: nth($item, 1);

package/scss/library/_triangle.scss

@@ -36,6 +36,6 @@
       @content;
     }
   } @else {
-    @error "The argument for direction must be one of the followings: #{quote($list-of-directions)}";
+    @error "The argument for direction must be one of the followings: #{$list-of-directions}";
   }
 }

package/scss/utilities/_clear-whitespace.scss

@@ -1,6 +1,9 @@
 @charset "UTF-8";
 
 @function __clearWhitespace($string) {
+  @if type-of($string) != "string" {
+    @error "`#{$string}` is not a valid $string for `__clearWhitespace`. Pass a string.";
+  }
   @while (str-index($string, " ") != null) {
     $index: str-index($string, " ");
     $string: "#{str-slice($string, 0, $index - 1)}#{str-slice($string, $index + 1)}";

package/scss/utilities/_convert-to-number.scss

@@ -2,6 +2,10 @@
 
 @function __convertToNumber($value) {
 
+  @if type-of($value) != "string" {
+    @error "`#{$value}` is not a valid $value for `__convertToNumber`. Pass a string made only of digits, such as `\"42\"`.";
+  }
+
   $string-numerals: "0" "1" "2" "3" "4" "5" "6" "7" "8" "9";
   $calc: 0;
 

package/scss/utilities/_null.scss

@@ -1,6 +1,9 @@
 @charset "UTF-8";
 
 @function __null($value, $seperation: comma, $skip: false) {
+  @if not index("comma" "space" "slash", $seperation) {
+    @error "`#{$seperation}` is not a valid $seperation for `__null`. Pass `comma`, `space` or `slash`.";
+  }
   $list: ();
   @for $i from 1 through length($value) {
     @if nth($value, $i) == null {

package/scss/utilities/_pixelify.scss

@@ -3,6 +3,9 @@
 // This function checks if the argument has a px unit. If not, it adds a px unit to the end of the argument.
 
 @function __pixelify($value) {
+  @if type-of($value) != "number" {
+    @error "`#{$value}` is not a valid $value for `__pixelify`. Pass a number, with or without a unit.";
+  }
   @if unit($value) == "px" {
     @return $value;
   } @else if unit($value) != "px" {

package/scss/utilities/_validate-length.scss

@@ -1,13 +1,25 @@
 @charset "UTF-8";
 
 @function __validateLength($value) {
-  $list: ("auto", "inherit", "initial", 0);
-  @if index($list, $value) {
+  $keywords: ("auto", "inherit", "initial", "unset", "revert", 0);
+
+  @if index($keywords, $value) {
+    @return $value;
+  }
+  @if type-of($value) == "number" and not unitless($value) {
     @return $value;
-  } @else if type-of($value) == "number" and not unitless($value) {
+  }
+  // calc(), clamp(), min() and max() arrive as Sass calculations; var() and
+  // env() arrive as unquoted strings holding a parenthesis. Both are valid
+  // lengths, and warning about them means warning about correct modern CSS,
+  // which this function used to do.
+  @if type-of($value) == "calculation" {
     @return $value;
-  } @else {
-    @warn "Please pass some values with unit like `#{$value}px`, `#{$value}em`, or `#{$value}%` etc. or pass one of the following arguments: `auto`, ìnherit`, ìnitial` or `0`.";
+  }
+  @if type-of($value) == "string" and str-index(quote($value), "(") {
     @return $value;
   }
+
+  @warn "`#{$value}` does not look like a length. Pass a number with a unit such as `20px`, a CSS function such as `var(--gap)` or `calc(1rem + 2px)`, or one of: auto, inherit, initial, unset, revert, 0.";
+  @return $value;
 }

package/scss/utilities/_validate-scissors.scss

@@ -3,6 +3,12 @@
 @function __validateScissors($value) {
   $list: ();
   @for $i from 1 through length($value) {
+    $item: nth($value, $i);
+    // unitless() below throws on anything that is not a number, and its message
+    // names its own parameter rather than the corner that was wrong.
+    @if $item != null and type-of($item) != "number" {
+      @error "`#{$item}` is not a valid corner size. Pass a length such as `12px`, or a unitless number which is treated as pixels.";
+    }
     @if nth($value, $i) == null {
       $list: append($list, 0px, comma);
     } @else if unitless(nth($value, $i)) {