Сразу скажу, что ты ж просил недостатки указать, а не хвалить) У меня первый сайт был в разы хуже.
Каталог товаров: Решение с классами left-center-right очень неудобное.
Представь, что у тебя таких товаров куча и берутся они из базы (то есть точное число ты не знаешь, выводятся в цикле) . Надо будет ставить костыль, чтоб задавать эти классы корректно.
А адаптивность? Тебе нравится им флоаты убирать?) Выглядит не круто.
Читай про
flexbox , это просто и универсально
Чтоб не было такой шляпы, как показал
Exploding на скрине, карточкам фиксируют высоту.
Для картинок - если картинка - фон, то
background-size: cover/contain;
Кстати, в данном случае всё таки я бы поставил тэг img , а не фон. И центрировал картинку через флексы или
position: relative
transform: translate(....)
Кнопка на видео.
Нельзя менять ей ширину и высоту. По многим причинам, можно погуглить их. В данном случае идеально подойдёт
transform: scale(...)
Теперь по главной:
"Коротко о нас" - загуглить таки flexbox и не делать костыль из двух списков больше никогда))
position: absolute для таких целей надо стараться не применять, это плохая практика как-минимум из-за адаптивности
Я смотрю, что у тебя почти все svg позиционированны отступами и абсолютом. Не надо так, серьёзно.
Обёртка
div.wrapper - не знаю, зачем она тебе, но... пусть чтоли main.wrapper будет. И логичнее и html5
Простой
Средний