From bc64ffc498effe235687be6374e3a4a98d75d1ee Mon Sep 17 00:00:00 2001 From: JohnAlbin Date: Mon, 8 May 2017 20:44:59 +0800 Subject: [PATCH] Prevent unnecessary $normalize-vertical-rhythm usage. Fixes #97 --- fork-versions/default/_normalize.scss | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fork-versions/default/_normalize.scss b/fork-versions/default/_normalize.scss index a107a23f8..aafaa7458 100644 --- 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 */