1. Тег header напихан везде где только можно, хотя он используется для задания шапки сайта или основного заголовка страницы, но никак не для каждой статьи и сайдбара; 2. БЭМ - видимо не интересен; 3. Вместа тега section решено было использовать p.article_text, что мне не ясно, если делается такой упор на html5; 4. Свойство border-radius поддерживается не во всех браузерах, почему не написаны аналогичные свойства для старых версий популярных браузеров; 5. PSD было лень качать, но даже с jpg видно, что копирайт находится не на том месте, что в макете; 6. Нет ховеров на ссылки и на кнопки, плюс кнопка в форме подписки не того цвета; 7. При маленьком разрешении или окне браузера бекграунд под меню и страницей уползает влево в итоге половина страницы на белом фоне; 8. Не ясно почему в css-файле символы многострочных комментариев используются для одной каждой строки, например в блоке #header_nav span; 9. background: url(" - где то написан с кавычками, где то без, как-будто писали разные люди; 10. Когда для одного элемента одновременно применяются стили типа font-style: italic; font-size: 26px; line-height: normal; - то наверно есть смысл объединить их в одну запись; 11. Нет placeholder-а наинпуте, раз уж спользуется html5 можно было бы и побольше фишек добавить;