From 29d0a869405b780e869b45bc10eccd399f3f2d89 Mon Sep 17 00:00:00 2001 From: Krister Kari Date: Wed, 8 Feb 2012 00:49:09 +0200 Subject: [PATCH] Remove `hr` and separate some margin normalizations Recommend that people supporting IE6/7 do not use the `hr` element at all. It requires far more work than just normalizing margins. IE6/7 do not collapse margins set on `hr` with margins of pre- or proceeding elements. Separate the margin normalizations for `p` and `pre` from that for `h3`. Despite sharing the same margin value, headings are qualitatively different from these other elements. Both in terms of customization and debugging using browser tools, it is cleaner to keep the margins of heading separate. --- normalize.scss | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/normalize.scss b/normalize.scss index 81b78f8..b69f70e 100644 --- a/normalize.scss +++ b/normalize.scss @@ -114,11 +114,6 @@ h2 { h3 { font-size: 1.17em; -} - -h3, -p, -pre { margin: 1em 0; } @@ -154,12 +149,6 @@ blockquote { margin: 1em 40px; } -// Addresses styling not present in IE6/7 - -hr { - margin: 0.5em auto; -} - // Addresses styling not present in S5, Chrome dfn { @@ -173,6 +162,13 @@ mark { color: #000; } +// Addresses margins set differently in IE6/7 + +p, +pre { + margin: 1em 0; +} + // Corrects font family set oddly in IE6, S4/5, Chrome // en.wikipedia.org/wiki/User:Davidgothberg/Test59