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