Goreska

Проблема с логотипом, и конструктивная критика по сайту

Рекомендованные сообщения

https://artrises.github.io/index.html

Всем добрый день, это моя первая работа, прошу конструктивно описать какие проблемы есть, какие ошибки допустил (хочу не вырабатывать говнокод), а постепенно реализовывать себя как толкового фронтендера. Поэтому будет важно все - мои косяки, ваши предложения, огрехи которые вы увидели в работе. И прошу помочь с одной проблемой - логотип в шапке - под ним как будто маленький маргин\паддинг, но я обнулил его, через инспектор тоже не нашел в чем проблема, какая то брешь между логотипом и контейнером оберткой.

Заранее спасибо.

Изменено пользователем Goreska

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

1. Траблы с семантикой, телефоны в теге абзаца, коли есть header, footer, где section? Встречаются лишние блоки обертки...
2. Адаптив... Юзаете адаптивную сетку бутстрапа, но адаптива там нет совсем... К тому же col должны быть в row!
3. Не мешало бы оживить и карусель.
4. Картинка в лого прыгает по причиние того, что блок в картинкой имеет меньшую высоту, чем соседние блоки, который тянут фон шапки...
5. Я бы не использовал отрицательные маржины, в роли костылей... Есть моменты где они нужны, в тех же сетках, и т.д. Но в вашем случае это костыль...

Учим семантику, разбираем принцип адаптивных сеток...
По себе знаю, что на первую работу это совсем не похоже))) Ибо в действительно первых работах такие трешаки, что через 2-3 года смотришь со слезами))) На Ваше добро можно смотреть без слез)))

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

Создайте аккаунт или войдите в него для комментирования

Вы должны быть пользователем, чтобы оставить комментарий

Создать аккаунт

Зарегистрируйтесь для получения аккаунта. Это просто!

Зарегистрировать аккаунт

Войти

Уже зарегистрированы? Войдите здесь.

Войти сейчас

  • Похожие публикации

    • Автор: adriano31
      Rebeata, nujna pomosh. Ne znaiu kak skachat mnogo jpg files odnim klick-om . 
      Primer :
      http://img.tennis-warehouse.com/new_big/AMDBNBW-1.jpg
      http://img.tennis-warehouse.com/new_big/AMDBNBW-2.jpg
      http://img.tennis-warehouse.com/new_big/AMDBNBW-3.jpg
      http://img.tennis-warehouse.com/new_big/AMDBNBW-4.jpg
      http://img.tennis-warehouse.com/new_big/AMDBNBW-5.jpg
       
      Jelatelno bez zip, no esli netu drugogo vyhoda...
       
      Izvinite za moi ruskii. 
       
       
    • Автор: umid99
      Доброго всем времени суток,  нужна ваша помощь .  Нужно сделать небольшое всплывающее окно на лэндинг которое показывает сообщения типа "Сейчас марина купила товара на сумму ххх " или "Осталось товара по акции;5шт" , по ссылке пример. Желательно сделать 1в1 . всех благодарю за помощь !