🐛 Don't define mixin within a conditional

stormwarning · Aug 20, 2016 · cde10fc · cde10fc
1 parent 9a0265b
commit cde10fc
Showing 1 changed file with 21 additions and 18 deletions.
diff --git a/utilities/_ampersand.scss b/utilities/_ampersand.scss
@@ -1,16 +1,25 @@
-@if $typ-use-opentype == true {
+//
+// Custom ampersand font-face.
+//
+// Set ampersands in a different typeface using `unicode-range`.
+//
+// @link 24ways.org/2011/creating-custom-font-stacks-with-unicode-range
+// @package Typeset.css
+//
+$typ-ampersand-face:     Ampersand !default;
+$typ-ampersand-source:   local("Georgia"),
+                         local("Garamond"),
+                         local("Palatino"),
+                         local("Book Antiqua") !default;
+$typ-ampersand-fallback: local("Georgia") !default;
+
+
+@mixin typ-ampersand($font-family...) {
+  font-family: $font-family;
+}
 
-  //
-  // Custom ampersand font-face.
-  //
-  // Set ampersands in a different typeface using `unicode-range`.
-  //
-  // @link 24ways.org/2011/creating-custom-font-stacks-with-unicode-range
-  // @package Typeset.css
-  //
-  $typ-ampersand-face:     Ampersand !default;
-  $typ-ampersand-source:   local("Georgia"), local("Garamond"), local("Palatino"), local("Book Antiqua") !default;
-  $typ-ampersand-fallback: local("Georgia") !default;
+
+@if $typ-use-opentype == true {
 
   //
   // Ampersand `@font-face` setup.
@@ -27,12 +36,6 @@
     unicode-range: U+270C;
   }
 
-  @mixin typ-ampersand($font-family...) {
-
-    font-family: $font-family;
-
-  }
-
   //
   // Apply ampersand font-family by class or `@extend`.
   //