Привет.
В первую очередь лучше просить обратную связь у компании.
Что меня смущает:
– почему не смиксовать классы header__wrapper и container (то же самое в футере)
– какой-то странный нейминг menu__item__link, news__item__subheader и другие: двумя нижними подчеркиваниями в данном примере разделяются элемент и блок, скорее всего подразумевалось menu__item-link (в других вариантах так же)
– title_main и title_news очень разные сущности, они не подходят под модификатор блока title
– "Новости" точно заголовок. Максимум p или span, но никак не div
– аналогично везде, где текст: div не имеет семантику, а просто блочная обертка. Для текста используются соответствующие теги: h1-h2, p, i, b и т.д.
– карточку можно сделал отдельным блоком, а так же использовать тег article
К чему еще можно придраться
– "Лидер в разработке..." мне кажется больше на параграф тянет, чем на h1
– возможно ожидали, что картинка под "Лидер в разработке..." должна прописываться в html
– могли не понравиться орфографические ошибки (например, комментарии)
– лучше использовать текст из макета