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

[chiburator]

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

Жду конструктивную критику по html/css коду, на что обратить внимание, что улучшить.

[chiburator]

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

Жду конструктивную критику по html/css коду, на что обратить внимание, что улучшить.

Немного доработал основную страницу и добавил еще одну работу.

[br3t]

Не, ну если вы таким лендинг-пейджем планируете искать клиентов, то... мне кажется, нужно что-то менять. И, думаю, потенциальному клиенту больше понравятся НТМЛ5 и CSS3

Сайт, который IntArnational оценить трудно, слишком прост

По кас-веду - ни к месту употребляете br и заголовок в сайдбаре, кажется, лучше сделать заголовком

[alexforce2]

вместо

background-color: #FFCC66;
background-image:url("../i/bg-body.jpg");
background-repeat:repeat-y;
background-position:50% 0; 

такие правила лучше заменить на сокращённые, для экономии времени и размера файла

background:#FFCC66 url("../i/bg-body.jpg") repeat-y 50% 0;

зачем нужен данный код

#wrapper2 {
    border-left: 0 solid #FFFFFF;
    border-right: 0 solid black;
}

всё равно бордеры не показываютя?

Edited March 25, 2012 by alexforce2

[alexforce2]

в меню в хедере вместо

    margin-right: 20px;
    padding-left: 20px;

достаточно применить

padding:0 20px;

чтоб не плодить лишний код.

в блоке с контентом и в меню в сайдбаре вы используете тег <br> для задания отступов, а это не правильно. Используйте маргины для задания отступов.

[chiburator]

Не, ну если вы таким лендинг-пейджем планируете искать клиентов, то... мне кажется, нужно что-то менять. И, думаю, потенциальному клиенту больше понравятся НТМЛ5 и CSS3

Полностью с вами согласен, я совсем недавно встал на этот путь. Хочу сначала разобраться с HTML 4, все по пордяку .

У меня мало опыта, а криво верстать грамотные шаблоны я не буду . Нужно еще разбираться и многому учиться.

По кас-веду - ни к месту употребляете br и заголовок в сайдбаре, кажется, лучше сделать заголовком.

А как можно грамотно отформатировать с помощью CSS меню в <ul> .... </ul> без использования <br>?

В моем случае код такой

<div class="punkty-vert-menu">
<ul>
<li><a href="/">Skimmed milk powder</a></li><br><br>
<li><a href="/">Whole milk powder</a></li><br><br>
<li><a href="/">Whey powder</a></li><br><br>
<li><a href="/">Pasta</a></li><br><br>
<li><a href="/">Spaghetti</a></li>
</ul>
</div>

вместо

background-color: #FFCC66;
background-image:url("../i/bg-body.jpg");
background-repeat:repeat-y;
background-position:50% 0; 

такие правила лучше заменить на сокращённые, для экономии времени и размера файла

background:#FFCC66 url("../i/bg-body.jpg") repeat-y 50% 0;

Спасибо, принял к сведению.

Edited March 25, 2012 by chiburator

[ablay2009]

.punkty-vert-menu li{display:block;margin-bottom:20px;}

Edited March 25, 2012 by ablay2009

[NeoXidizer]

.punkty-vert-menu li {
list-style: none;
margin-bottom: 15px;}

используйте элементы по назначению

<br>, от слова break, в контексте html кода можно перевести как "разорвать"

например в данном коде <br> разрывает текст

Лето в стране настало,
Вереск опять цветет.
Но некому готовить
Вересковый мед.

который отобразится так

Лето в стране настало,

Вереск опять цветет.

Но некому готовить

Вересковый мед.

<br> разорвал единую строку, создавая переводы на новую линию

немного опоздал

Edited March 25, 2012 by NeoXidizer

[chiburator]

.punkty-vert-menu li{display:block;margin-bottom:20px;}

Теперь дошло, спасибо .

[br3t]

Бтв, я имел ввиду нецензурную лексику