Что неправильно в верстке?
Из глобального:
1. Страница весит 1.6мб. Это жуткая жуть для такой страницы. Зачем все картинки в png? Он там не нужен. Переводите всё в jpg и оптимизируйте jpg – выигрыш в весе будет огромный.
2. Поддержка ретины. Сейчас это становится уже важным. Все иконки лучше перевести в svg. Или, как мининмум, сделать png@2x. Но вектор лучше, с ним удобнее работать и весит он порой меньше png.
3. Зачем там 2 файла стилей и несколько спрайтов? + стили правильнее минифицировать.
Сама верстка:
1. Почитайте про различные методологии. БЭМ, например. Селекторы типа .site-header .site-menu li a — боль.
2. Блок с логотипами. Вставить всё одной картинкой это, конечно, легко и просто. Но заказчик наверняка бы попросил сделать каждый логотип ссылкой на сайт. Или, например, когда понадобится добавить еще один логотип. добавлять его будет больно.
3. Для ссылок можно тоже transition добавить. Хуже от него не будет.
Это так, главное, что бросилось в глаза :)
Я не знаю о каких ошибках вы говорите, но у меня в хроме открылось норм. Значит нормальная вёрстка.
Теперь пройдите курсы, как сделать всё это дело адаптивным.