Уважаемые,оцените верстку Винной страницы :)

[Temik2704]

Добрый день!

Я В общем-то начинающий,и хотелось бы услышать мнение о моей верстке,прошу высказывать то,что вам не нравится,постараюсь исправить!

Макет

Мой результат

Слайдер должен работать на JS,но его пока не изучал,в частности jQuery,поэтому оставил так.

 

P.s.Сколько бы вы затратили времени на создание данного макета?

[klierik]

1.

<div class="clear"></div>

старый метод.

 

2. 

плохо

.new-products-block ul li>span

хорошо:

.new-products-block .price {}

3.

плохо

.special-block ul li .text-block span.ex-price

очень трудоёмкое применение стилей браузером

1. в начале он найдет все .ex-price
2. потом все опросит на имя тега <span>

3. потом просканирует каждую ноду дерева и будет опрашивать каждую на наличие .text-block

4. потом просканирует все дерево и будет опрашивать селектор <li>

5. и так далее...

 

если глубина стилей больше 3х узлов -- ты скорее всего что-то делаешь не так.

 

4.

#footer ul.general-ul>li.contacts a.geo:hover:before

а вот тут при наведении картинка мигает, так как требуется время что бы обратится к серверу и загрузить её.

 

5.

http://wine.esy.es/styles/styles.css

ну тут все как обычно -- лучше переписать все с нуля нежели поддерживать такие стили (нечитабельно, не отформатированы)

 

оу, а что это такое?

/* Делаем 2 блока Ð´Ð»Ñ ÐºÐ°Ñ€Ñ‚Ð¸Ð½ÐºÐ¸ чтобы отцентровать по вертикалии горизонтали,а также задать max и min параметры */.new-products-block ul li .center-aligned {   display: table;  background: #FFFFFF;  width: 100%;  height: 210px;  border-radius:4px;  overflow:hidden;  margin-bottom: 10px;}

неуж-то там русский?

 

6. логотип не ссылка, а должен быть ею.

[Temik2704]

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

1,2,3,6 переделаю!

Насчет 4:Как лучше сделать чтобы не мигало?Может лучше выставить еще задержку в transition?Если нет - то как можно исправить?

Насчет 5:Скажите пожалуйста - что означает "читабельно"?Читабельность появляется после обработки каким-нибудь css сервисом форматировани?

Не отформатировано - обозначает отсутствие отступов для каждого уровня вложенности?Если да,то извините,думал данное правило распространяется только на html.

Комментарии лучше писать или на англ или транслитом?если русский превращается в крякозябры.

[Temik2704]

up

Хотелось бы услышать насчет 4 и 5 пунктов

[Struggle]

Слайдер так и должен оставаться не рабочем ?

[Temik2704]

Слайдер так и должен оставаться не рабочем ?

Да,т.к. до jQuery не добрался еще