Jump to content

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


DenChik
 Share

Recommended Posts

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

Link to comment
Share on other sites

Отлично для первого раза 'я щетаю'.

Одна ошибка по моему мнению точно есть. В левой колонке по логике один список, с перечислением проблем страхования или что то подобное, вы использовали три списка UL.

Лишняя разметка тут и там, но я не берусь утверждать что это ошибки. пример серый баннер под меню, стили со span-а можно перевесить на <div class="info_banner"> :


<div class="info_banner">
<span>Relax. ...
</span>
</div>

На кнопках тоже лишнего кода нафигачено. Стили с line_for_button можно на ::after ссылки перевесить. :


<div class="button_style">
<div class="line_for_button"></div>
<a href="#">Free Credit Report When You Sign Up!</a>
</div>

Link to comment
Share on other sites

Спасибо за замечания. Все исправил. Но со списками я честно говоря не понял, как используя один ul сделать левую границу второго столбца на одной линии.

Я вижу один способ:

li-шкам выставить ширину(width: 85px;),

#content_left li:last-child сделать 100% ширину,

с ul наоборот снять ограничение по ширине.

Тогда элементы выстраиваются в том порядке как у вас было впервоночальной верстке с тремя списками.

Link to comment
Share on other sites

Я вижу один способ:

li-шкам выставить ширину(width: 85px;),

#content_left li:last-child сделать 100% ширину,

с ul наоборот снять ограничение по ширине.

Тогда элементы выстраиваются в том порядке как у вас было впервоночальной верстке с тремя списками.

Спасибо все изменил, только поставил ширину (width: 105px;),так как при изменении масштаба сайта при ширине 85 весь список скачет.

<div class="head">

делай сразу html5 и не заморачивайся, раз поставил

<!DOCTYPE HTML>

Будьте добры, объясните. Не понял что делает див с классом head

Link to comment
Share on other sites

  • 2 weeks 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.

 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