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

[Crushing]

https://uzinouzi.github.io/Revelation/

Учебный PSD макет, что можете сказать по поводу верстки? Буду очень благодарен за конструктивную критику

[IvanRu08]

Пропишите вьюпорт для мобильных устройств. Почему то не работает слайдер + никак не реагируют иконки социальных сетей на наведение. Вместо reset.css рекомендую использоваться normalize.css. Не обязательно все стили сбрасывать. 

На странице должен быть только один тег H1, а у вас там их штук 9. Оставьте один h1, а остальные поменяйте на h2. В слайдере при наведении на кнопку Read more, она визуально увеличивается из за border. Поэтому добавьте к этой кнопке в обычном состоянии:

border: 2px solid transparent

Все, что заметил при беглом взгляде

[alanreid]

1. Двум потомкам блока main-slider задан правый и левый margin (по одному на каждый), вместо свойства padding по сторонам для родителя.

2. Слишком много container блоков. Для данной задачи достаточно одного для footer элемента и другого для остальных.

3. Слишком мало контрольных точек для media выражений. Не хватает внутренних отступов (padding) секциям - это заментно, когда ширина экрана равна ширине container элемента (960px, примерно, с учетом вертикальной полосы прокрутки). Так как ширина элементов в одном ряду задана в абсолютных единицах, а не в процентах, то виден зазор справа в блоке facilities и footer. Также, сетка в этих блоках ломается при ширине в 918px. В добавок, элементы этих блоков и блока services выстраиваются в столбец, в то время как вокруг них полно места. При ширине в 697px, навигация уходит под logo.

4. По семантике тоже есть пара недочетов. Блоки block_facilities, block_services, footer_block, по хорошему, section элементы, а не div. Что прослеживается по outline схеме.

И т.д.

[Crushing]

8 часов назад, IvanRu08 сказал:

Пропишите вьюпорт для мобильных устройств. Почему то не работает слайдер + никак не реагируют иконки социальных сетей на наведение. Вместо reset.css рекомендую использоваться normalize.css. Не обязательно все стили сбрасывать. 

На странице должен быть только один тег H1, а у вас там их штук 9. Оставьте один h1, а остальные поменяйте на h2. В слайдере при наведении на кнопку Read more, она визуально увеличивается из за border. Поэтому добавьте к этой кнопке в обычном состоянии:

border: 2px solid transparent

Все, что заметил при беглом взгляде

 

5 часов назад, alanreid сказал:

1. Двум потомкам блока main-slider задан правый и левый margin (по одному на каждый), вместо свойства padding по сторонам для родителя.

2. Слишком много container блоков. Для данной задачи достаточно одного для footer элемента и другого для остальных.

3. Слишком мало контрольных точек для media выражений. Не хватает внутренних отступов (padding) секциям - это заментно, когда ширина экрана равна ширине container элемента (960px, примерно, с учетом вертикальной полосы прокрутки). Так как ширина элементов в одном ряду задана в абсолютных единицах, а не в процентах, то виден зазор справа в блоке facilities и footer. Также, сетка в этих блоках ломается при ширине в 918px. В добавок, элементы этих блоков и блока services выстраиваются в столбец, в то время как вокруг них полно места. При ширине в 697px, навигация уходит под logo.

4. По семантике тоже есть пара недочетов. Блоки block_facilities, block_services, footer_block, по хорошему, section элементы, а не div. Что прослеживается по outline схеме.

И т.д.

Спасибо большое

[rpmcmerphy]

мне кажется все таки где меню HOMEPAGE навигация должна сворачиваться)