Jump to content

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


Recommended Posts

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

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

border: 2px solid transparent

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

  • Thanks 1
Link to post
Share on other sites

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 схеме.

И т.д.

  • Thanks 1
Link to post
Share on other sites
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 схеме.

И т.д.


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

Link to post
Share on other sites
  • 2 months later...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy