Оцените верстку

Общее
1

Здравствуйте, оцените пожалуйста верстку bikes https://github.com/Slavamel/develop.git

Посоветуйте, куда лучше выкладывать свои сверстанные макеты, для оценки, и для работодателя?

1 лайк
2

Cедона

  1. Логотип надо бы вынести из навигационного меню и спозиционировать по хедеру внутри. Соответственно у второго пункту надо сделать отступ справа 20 процентов.
  2. http://take.ms/O5Ndn лучше стили вешать не по семантических элементам а по классу. Добавить класс например main-header
  3. Блок main тут не несет никакой функции, кроме того что в IE с ним будут проблемы.Лишняя обертка.
  4. <q> ? он уже устарел используется blockquote
  5. http://take.ms/f6bQU маску если так хочется сделать отдельным элементов то лучше псевдоэлемент чем пустой блок. А лучше вообще множественным фоном на большой блок.
  6. Зачем тут br если можно ограничить ширину. http://take.ms/Ifhp6
  7. В блоке с картинкой картинка не задним фоном идет, а через img
  8. http://take.ms/upOw2 второй класс не используется
  9. http://take.ms/K4QcT тут лучше будет так. В верхнем блоке заголовок, текст и кнопка которая открывает и закрывает форму. А вот форма уже внутри блока с картой. И не надо будет развлекаться с relative
  10. И футер внести в блок с картой и не надо будет ничего позиционирвоанием делать.
  11. На плюс минус не хватает курсора в виде ручки
  12. Все иконки либо в иконочный шрифт либо спрайт надо делать
  13. Странное решение, http://take.ms/4J2ic проще было всему блоку задать отступ чем каждой иконке.
  14. Блоки должны идти в том порядке в котором они на макете) http://take.ms/zLkAV
    Тут примерно такая сетка должна быть http://take.ms/87XKd
  15. Какие то инлайовые стили http://take.ms/VNShH это недопустимо нарушение валидности.
  16. Форма нулевой высоты http://take.ms/1XKOQ распорка не там стоит. Внутри формы нарушен порядок блоков. Опять первый самый правый.
  17. С этой штукой что то случилось http://take.ms/WlTpU
  18. В одной строке инлайны, флоуты, вообще без всего строчные элементы) http://take.ms/DTEkb Определитесь

Велосипеды

  1. Нарушен порядок поключения http://take.ms/z9FrR
    Сначала шрифты, потом нормалайз, потом ваши стили. JS в самый низ.
  2. Less я бы не стал компилить на стороне клиента, лучше ставить готовый css
  3. http://take.ms/eHvmy активная ссылка лезет из блока
  4. http://take.ms/WxWT0 дак абсолют или инлайн блок?) и опять смесь всего в одной строке.
  5. http://take.ms/wvala это не картинки заднего фона, это img
  6. http://take.ms/O9fGy не там оказалось
  7. http://take.ms/TwXBQ - псевдоэлемент
  8. http://take.ms/yv7nb это свойство не работает)
  9. http://take.ms/iEs3W только слово SALE фон, остальное цвет, а треугольник псевдоэлемент
  10. Какие то манипуляции с позиционирвоаем в потоке http://take.ms/QlERh
  11. Очень страшные селекторы http://take.ms/X0rFM . Вы переигрываете с less следите за вложенность в 2 объекта. У вас все зависит от main если я его уберу все сломается.
  12. СТрелочка это абсолютный блок ко всему блоку http://take.ms/iddGq
  13. http://take.ms/etGIY вылезает и паддинг слева зачем то
6 лайков
3

все забываю поблагодарить тебя за оценку)) Спасибо большое)

1 лайк