@Zimniros

Оцените качество верстки и дайте советы новичку. Какие грубые ошибки(кроме шрифта) в нем присутствуют?

Добрый вечер. Кому не сложно, оцените качество верстки(это мой первый макет).

Онлайн:
antonweb.github.io

Сам макет:
https://yadi.sk/d/z2qfzcRxmCLqx

p.s. Шрифта в интернете не нашел, поэтому загрузил его локально.
  • Вопрос задан
  • 329 просмотров
Решения вопроса 1
27cm
@27cm
TODO: Написать статус
p.s. Шрифта в интернете не нашел, поэтому загрузил его локально.

А чтобы установить его локально, вы же его откуда-то взяли?
Используйте www.fontsquirrel.com/tools/webfont-generator

Кстати, шрифт есть там же:
www.fontsquirrel.com/fonts/montserrat

Тянуть весь Font Awesome ради нескольких иконок - перебор. Используйте сервис вроде: fontello.com
Кстати, иконки у вас отличаются от тех, что в макете.

В макете - всё разложено по сетке, у вас - нет.
https://habrastorage.org/files/314/ef1/09d/314ef10...
Пользуйтесь хотя бы bootstrap'овской сектой: bootstrap-ru.com/scaffolding.php

Вместо сброса стилей (reset) используйте normalize.css:
stackoverflow.com/questions/6887336/what-is-the-di...

Красные точки я бы сделал через всевдоэлемент :after. У вас слишком уж большой селектор получается для красной точки. Представьте, что вы не одну бы страницу верстали, и блоков с точками было бы не четыре, а сорок четыре.
.hero-text span,
.features-item span,
.portfolio-column-right span,
.team-section span {
  color: #fa5c65;
}


UPD:
Есть лишние теги: вместо <a href="#"><i class="fa fa-twitter"></i></a> можно было писать <a href="#" class="fa fa-twitter"></a>.
Неиспользуемый класс portfolio-section.
Ответ написан
Пригласить эксперта
Ответы на вопрос 1
customtema
@customtema
arint.ru
Не адаптивно. Тривиально.

Есть смысл в такой верстке без адаптивности? Если это лендинг, тогда по вертикали можно выровнять.

У меня отображается корректно (Firefox 42.0 for Ubuntu).
Ответ написан
Комментировать
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы