]> git.ipfire.org Git - thirdparty/bootstrap.git/commitdiff
Escape SassScript in custom variable values (#24354)
authorGleb Mazovetskiy <glex.spb@gmail.com>
Sat, 14 Oct 2017 07:29:43 +0000 (08:29 +0100)
committerGitHub <noreply@github.com>
Sat, 14 Oct 2017 07:29:43 +0000 (08:29 +0100)
This is required for Sass v3.5+ compatibility.
See https://github.com/sass/sass/blob/stable/doc-src/SASS_CHANGELOG.md#backwards-incompatibilities----must-read and https://github.com/sass/sass/issues/2383

scss/_root.scss

index 78ac03915db8ca0dbc4eca8eea96b47ddfd14a2c..ad550df3b49b5374b82c0266a00a9c6e416b6155 100644 (file)
@@ -1,16 +1,19 @@
 :root {
+  // Custom variable values only support SassScript inside `#{}`.
   @each $color, $value in $colors {
-    --#{$color}: $value;
+    --#{$color}: #{$value};
   }
 
   @each $color, $value in $theme-colors {
-    --#{$color}: $value;
+    --#{$color}: #{$value};
   }
 
   @each $bp, $value in $grid-breakpoints {
-    --breakpoint-#{$bp}: $value;
+    --breakpoint-#{$bp}: #{$value};
   }
 
-  --font-family-sans-serif: $font-family-sans-serif;
-  --font-family-monospace: $font-family-monospace;
+  // Use `inspect` for lists so that quoted items keep the quotes.
+  // See https://github.com/sass/sass/issues/2383#issuecomment-336349172
+  --font-family-sans-serif: #{inspect($font-family-sans-serif)};
+  --font-family-monospace: #{inspect($font-family-monospace)};
 }