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

Общее
1

Оцените первую мою верстку сайта. Верстал без JS, жду адекватной аргументированной критике по коду. Вот ссылка на верстку

Макет по которому я верстал:

marble-min.png1600×2306 489 KB

2

  • “Слайдер” не стоило засовывать в хедер, я сомневаюсь что на всех страницах этого сайта (если бы они были) он будет. Хедер и футер - общие блоки для всех страниц сайта, они не меняются от страницы к странице.

  • Когда подключаете шрифты с разной жирностью, не надо для каждого делать отдельную font-family с bold или light в конце. font-family должен быть один, а разную жирность указывать в параметре font-weight подключаемого шрифта

1 лайк
3

А вообще слайдер засовывают в хедер? Насчет шрифтов я немного подзабыл, что так нужно делать. Спасибо, за вашу критику.

4

Как я уже говорил хедер используется для тех блоков которые будут на всех страницах сайта.
Обычно это лого, меню, может ещё что нибудь.

Если у вас слайдер по макету на всех страницах сайта, то отчего и не засунуть.

5

Теперь стало все понятно. Спасибо еще раз.

6
  • font-family: “RobotoRegular”, sans-serif;
    Принято помимо семейства шрифта, указывать ещё и фоллбэк шрифт, если основной не загрузится, например Arial.
    Список “безопасных” шрифтов можно посмотреть тут - http://www.cssfontstack.com/
7

Спасибо, просто про запасные шрифты подзабыл вот и не указывал.