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

[djan]

Вот объект собственно http://djan.esy.es/herbal/HerbalMedicineCompany.htm'>http://djan.esy.es/herbal/HerbalMedicineCompany.htm
Вот макет

Edited November 4, 2013 by djan

[by chris]

body {

background: -moz-linear-gradient(center bottom , #7CD034 0%, #6EC23E 74%, #076386 95%) repeat scroll 0 0 rgba(0, 0, 0, 0);

min-height: 100%;

height: 100%;

}

.Wrapper {

background: -moz-linear-gradient(center bottom , #7CD034 0%, #6EC23E 74%, #076386 95%) repeat scroll 0 0 rgba(0, 0, 0, 0);

}

[binomer]

Только ради того, чтобы что-то сказать Ты одну иконку в футере пропустил.

[advokatua]

1) блок .logo пустой .

2) Зачем эти непонятные белые полоски в правой части верхнего меню?

3) Бекграунд должен растягиваться влево и вправо на всю ширину экрана.

4) Зачем пунктам меню задавать фиксированную ширину?

5) Картинки в Продуктс и остальных нужно пихать в html, т.к. они могут меняться периодически.

6) - Заголовок анонса лучше обрамлять в <h$>.

- Для дат есть специальный тег (гуглим).

<div class="SkinText"><b>The artical below explains</b>
		 <span class="dateSkin">Monday,13 June 2011</span><p>
		 The article below explains how nettles works in the body against the inflammatory repring is in the air and I ant hearing
			 many people com plain of seasonal allergies. Nettle extract can be a key component. </p>
		 <p>Nettle extract can be a key component in your fight against seasonal allergies... </p>

7) Нет никакого смысла ставить инлайн-бокс, для абсолютно поз. элемента, как тут. Впрочем как и ставить абсолютную позицию, а затем выравнивать блок марджином.

.SkinText {
margin: 145px 28px;
width: 325px;
height: 105px;
font: 0.62em sans-serif;
line-height: 14px;
display: inline-block;
position: absolute;
}

8) Меню абсолютно поз., class="content-welcome" также в абсолюте. Так быть точно не должно.

9) Почти везде неверно выбрана гарнитура шрифта. Там где должна быть с засечками выбрана без, а где нужно без выбрано с.

Косяков полно. Советую почитать замечания в остальных темах данного раздела. Буквально не менее половины тебе также подойдет.

Учиться, учиться, мучиться

Edited October 26, 2013 by advokatua

[cris7iano]

Высота wrapper не нужна,

непонятные названия классов

пустые блоки это плохо-для cms'ки.

картинки в content-bottom clearfix вставляются в html, это не часть дизайна, а контент

соц.сети делать нужно спрайтом 100 раз об этом писал.

FootText1-FootText2-FootText3-FootText4-почему разные классы???ведь все 4 блоки одинаковы.

[djan]

Поисправлял немного. 

 

 

Зачем эти непонятные белые полоски в правой части верхнего меню?

Так изначально было в макете (скриншон случайно был зделан без него) - задача была зделать один в один с дизайном макета .

 

 

Для дат есть специальный тег (гуглим).

Спецификация документа XHTML а для него тэги типа header,footer,time,section,article крайне не валидны.

 

 

FootText1-FootText2-FootText3-FootText4-почему разные классы???ведь все 4 блоки одинаковы.

У них разная ширина.

 

 

Только ради того, чтобы что-то сказать   Ты одну иконку в футере пропустил. 

Если ты про знак копирайта - то я намеренно его не поставил.

Спасибо за критику.

Вот ещё - при увеличении-уменьшении кнопка в сообщении скачет вверх-вниз.

Как исправить?

[Temik2704]

Я новичок,но думаю что лучше сделать фон через url картинку с голубым градиентом и background position`ом разместить ее наверху,а остальное залить зеленым.

Типо такого.

 

background:url(gradient.png) repeat-x 0% 5%,#00FF00;

[djan]

Не вижу  смысла : подгружать картинку вместо нескольких строчек в css.