Prevent unnecessary $normalize-vertical-rhythm usage. Fixes #97

JohnAlbin · May 8, 2017 · bc64ffc · bc64ffc
1 parent a695a65
commit bc64ffc
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/fork-versions/default/_normalize.scss b/fork-versions/default/_normalize.scss
@@ -4,10 +4,8 @@
 @import 'vertical-rhythm';
 
 // If we've customized any font variables, we'll need extra properties.
-@if $base-font-size != 16px
-  or $base-line-height != 24px
+@if $base-line-height != 24px
   or $base-unit != 'em'
-  or $h1-font-size != 2    * $base-font-size
   or $h2-font-size != 1.5  * $base-font-size
   or $h3-font-size != 1.17 * $base-font-size
   or $h4-font-size != 1    * $base-font-size
@@ -29,10 +27,12 @@
 
 html {
   font-family: $base-font-family; /* 1 */
-  @if $normalize-vertical-rhythm {
+  @if $base-font-size != 16px or $normalize-vertical-rhythm {
     // Correct old browser bug that prevented accessible resizing of text
     // when root font-size is set with px or em.
     font-size: ($base-font-size / 16px) * 100%;
+  }
+  @if $normalize-vertical-rhythm {
     // Establish a vertical rhythm unit using $base-font-size and
     // $base-line-height variables.
     line-height: ($base-line-height / $base-font-size) * 1em; /* 2 */