Что скажете по поводу верстки?

1. h1, h2, h3, h4, h5, h6, p, a { display: inline-block; } - весьма неожиданно. Зачем так глобально переопределять?
2. Обнуление списков конечно удобно, но как быть с отображением списков? (обнулять нужно конкретные классы)

ну а так в целом норм.

Код не читаемый. Сравни:
1) Твой
2) Как должно быть

Если представить, что макет не твой и сайтом никто не будет пользоваться, то сделано не плохо. А иначе очень много совершенно очевидных проблем:
1) При наведении на звездочку, сайт должен говорить мне, что я на нее навел. Как минимум заливкой
2) Почему при наведении на кнопку Login, она подсвечивается, а аналогичная кнопка поиска - нет?
3) Так не должно быть. Блок должен занимать всю высоту
4) Для чего вообще outline?
5) Слишком много элементов с маленьким шрифтом, которые к тому же сливаются с фоном
6) Простой serif не подходит для кнопок, используй sans serif

И так далее. Но для начала довольно неплохо.