Правило дизайн не обсуждаю, но тут конечно дизайн просто финиш, очень жаль при оценке верстки тяжело отключить визуальную составляющую, и не понятно где косяки дизайна а где верстки.
Исхожу из того что это пиксель перфект :)
Если хочешь чтобы оценили верстку нужны исходники.
1.Кнопка 'Show me More'
class='btn get-tools__btn'
меняем на
class='btn btn--get-tools', все кнопки в отдельный файл.
2.Моя рекомендация, если есть container, и есть секции на 100% ширины экрана то лучше сделать для них класс container-fluid (отдаем должное Boostrap).
3.Тексты, заголовки и.т.д. Можно делать вложенные по БЕМ, или если страниц много и тексты в разных местах повторяются то
h2 class="collection__title"
меняй на
h2 class="text text--50 text--red text--lh-1"
4.Navbar не хватает ховеров, на флаги и на социальные ссылки.
5.Про отсут слева и выпадающие меню тебе уже сказали.
6. Слайдер не работает :(
7. При клике на элемент слайдера, появляется рамка, а почему у рамки сверху не закруглены края ?
8. quantity__events ховер срабатывает сразу не две кнопки, плюс и минус как то странно, и кнопки не работают.
9. CSS медиа лучше сразу писать к нужному элементу.
10.Скриптов мало, скрипты не сжаты.
Верстка по БЕМ в целом все хорошо, на медиа не разваливает.
Классы названы хорошо читаемо, есть контейнеры, ховеры, даже анимацию добавил ;), иконки через классы на бэкграунды, html/css сжат.
P.S. Ivan Bogachev хорошо расписал, что визуально сразу режет глаз.
