Jump to content

Верстка по макету


MadBanshee
 Share

Recommended Posts

Всем добрый день!

Оцените, пожалуйста, верстку стартовой страницы сайта по макету.

 

Макет

Страничка

 

Хотелось бы услышать критику касательно организации кода, адаптивности, ну и вообще какие ошибки бросаются в глаза.

 

Заранее спасибо!

 

Link to comment
Share on other sites

Что касается замечаний:

  • У Вас на десктопе почему-то скрыта навигация, хотя на превьюшке дизайна она есть (дизайн, кстати, скачать так и не удалось - ссылки битые). Диз: http://joxi.ru/Vrwl8BoIK1EP4m
    Верстка Ваша: http://joxi.ru/Vm67xOySxGlaQ2 .
  • Также я бы не задавал элементам стили какие-либо вот так: http://joxi.ru/EA4vzRNhD314Xm. Имеется ввиду именно хэдер. А что если вы будете использовать хэдер в каком-то блоке? И, например, нужно будет прижимать кверху его? Будете перебивать верхний марджин? 
  • Также лучше не писать вот так: http://joxi.ru/V2VKLNnH0bPjLA . А что если в контейнере понадобится использовать не section тэг, а див, например? Нужно будет лезть в цсс и менять название тэга на див? Лучше назначить класс каждому элементу, тогда не придется лезть в цсс и постоянно его править - не важно, какой элемент Вы используете, у него всегда будет этот класс с определенными стилями. Также проблема может возникнуть, если вы пожелаете вдруг вложить в эту секцию другую секцию, у которой стили унаследуются, что, скорее всего, будет не нужно и Вы снова будете их перебивать какими-то своими. В общем, лучше не используйте имена тэгов в качестве цсс-селекторов.


    Ну а в целом сверстано неплохо. Побольше практики и все будет отлично! Мой совет - попробуйте сверстать многостраничный сайт, покажите его друзьям и спросите, что бы они хотели улучшить в нем, чтобы Вам пришлось что-то переделать(а ведь в реальности клиент всегда просит что-то подправить), тогда вы прочувствуете все приведенные мной советы на практике.
     
Edited by yardim
Link to comment
Share on other sites

Спасибо большое за столь развернутый ответ и пожелания!

Да, со всеми замечаниями согласен, все по делу.

 

А что касается скрытой навигации, то действительно была серьезная ошибка (со скриптом накосячил), но теперь должно работать. Проверьте, пожалуйста, еще раз.

Link to comment
Share on other sites

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.

 Share

×
×
  • 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