Первое, что бросилось в глаза:
1. Зачем shop-block-1, shop-block-2, shop-block-3, и для каждого отдельно прописан стиль?
Вот здесь на месте бэкендера я бы точно придрался. Это одинаковые блоки, им не нужны разные классы.
2. <h5>В наличии: 3 шт</h5>
- семантически неверно, заголовок тут не нужен.
3. Несколько h1 заголовков
4. Навигация рассыпается, если ширина окна менее 1920px
для начала хватит.
P.S. Учитывая, что это ваша первая верстка, в целом нормально. Но до продакшена еще много допиливать.