В целом выглядит симпатично.
Все завернуто в тег main. это семантически неправильно.
Логотип внесен в общее меню, это вроде и не ошибка, но при интеграции с любой цмс будут проблемы.
Адаптив выглядит так, как будто мета тега viewport нет, хотя он есть.
Меню не прячется в бутерброд. Возможно вы скажете, что оно и так влазит, но это первый признак совсем начинающего, сам таким был.
Вы кое где используете сетку бутстрап, но не до конца понимаете как она работает, поэтому пишите лишние классы. например col-md-6 col-lg-6. В этом месте класс col-lg-6 лишний, так как делает ровно то же, что col-md-6, потому что бустстрап мобайл-фест. А там где одна колонка не нужен ни row ни col-12.
Не понял я смысл скроллов в карточках, выглядит намного хуже, как если бы где то было больше или меньше контента.
методАлогия))
