w <header> dał bym <h1> i <h2> zapmiast <p> i wstawił oba w znacznik <hgroup> (albo <h1> i w nim <small>)
<aside> powinien być w <main>
Stara przeglądarka IE ma problem z takim zapisaem
header.logo h1 {
margin: 0;
padding: 40px;
font-weight: 100;
}
Ona nie rozumie co to jest <header> i w tym wypadku CSS nie zadziała przeglądarka to zignoruje, jest na to banalny patent. Wystarczy że robiąc <header> nadasz mu classę "header". Będzie to wyglądało w następujący sposób:
<header class="header">...</header>
<main class="main">...</main>
i analogicznie w CSS
.header .logo h1 {
margin: 0;
padding: 40px;
font-weight: 100;
}
Dostawiasz tę kropkę i problem z głowy, nawet najgorsza przeglądarka zrozumie co ma robić :)
Tak po za tym to wygląda to całkiem nieźle, validator.w3.org nic nie pokazuje po za błędnym linkiem w "twitter".