From 8ea653ed3e5667788e934898cc40912f75f6c5d9 Mon Sep 17 00:00:00 2001
From: Dan Barber <dan@danbarber.me>
Date: Sun, 18 May 2014 20:22:58 +0100
Subject: [PATCH] Fix up header markup and CSS.

Works better in Firefox now.
---
 _assets/stylesheets/includes/layout.css.sass | 20 ++++++++++++--------
 _assets/stylesheets/includes/styles.css.sass |  6 +++---
 _includes/header.html                        |  6 +++---
 3 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/_assets/stylesheets/includes/layout.css.sass b/_assets/stylesheets/includes/layout.css.sass
index cb04e69..b9143b8 100644
--- a/_assets/stylesheets/includes/layout.css.sass
+++ b/_assets/stylesheets/includes/layout.css.sass
@@ -2,15 +2,19 @@ $max-width: 800px !default
 $pad-min: 8.5% !default
 
 header[role=banner]
+  padding: 0.52em 0 0.42em
   h1
     font-size: 3.43em
-    line-height: 0
-    color: white
-    margin: 0.52em 0 0.42em
-    text-shadow: 0 0 10px rgba(0, 0, 0, 0.2)
-    span
-      position: absolute
-      right: 100%
+    margin: 0
+    a
+      color: white
+      line-height: 0
+      text-decoration: none
+      text-shadow: 0 0 10px rgba(0, 0, 0, 0.2)
+      span
+        width: 0
+        font-size: 0
+        overflow: hidden
 
 article img
   max-width: 100%
@@ -24,7 +28,7 @@ article img.right
   float: right
   margin-left: 1em
 
-#content, hgroup, footer
+#content, .container, footer
   padding: 0 $pad-min
   margin: 0 auto
   max-width: $max-width
diff --git a/_assets/stylesheets/includes/styles.css.sass b/_assets/stylesheets/includes/styles.css.sass
index 7e4d1fd..dd9a262 100644
--- a/_assets/stylesheets/includes/styles.css.sass
+++ b/_assets/stylesheets/includes/styles.css.sass
@@ -18,11 +18,11 @@ body > header
   box-shadow: 0 4px 8px rgba(0, 0, 0, 0.2), inset 0 -1px 0px rgba(255, 255, 255, 0.4)
   padding-top: 0.8em
   padding-bottom: 0.9em
-  h1
-    display: inline-block
+  h1 a
+    //display: inline-block
     margin-top: 0
     font-weight: 700
-    line-height: 0
+    //line-height: 0
 
 a
   +transition(color 0.25s 0s ease)
diff --git a/_includes/header.html b/_includes/header.html
index d3f5fa4..9d02db6 100644
--- a/_includes/header.html
+++ b/_includes/header.html
@@ -1,6 +1,6 @@
   <header role="banner">
-    <hgroup>
-      <a href="/"><h1 class="icon-logomark"><span>Dan Barber</span></h1></a>
+    <div class="container">
       <nav role="navigation">{% include navigation.html %}</nav>
-    </hgroup>
+      <h1><a class="icon-logomark" href="/"><span>Dan Barber</span></a></h1>
+    </div>
   </header>