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

[Raura]

http://yuriy1983.ucoz.ua

[Galaxy]

Бегло просмотрел  но что бросилось в глаза : слишком много id - чем это обосновано? wrapper - это класс он может использоваться и не один раз у вас же он id, jquery надо юзать посвежее , избыточность css'a, скрипт который в body его лучше кидать в конец документа.

[Raura]

Бегло просмотрел  но что бросилось в глаза : слишком много id - чем это обосновано? wrapper - это класс он может использоваться и не один раз у вас же он id, jquery надо юзать посвежее , избыточность css'a, скрипт который в body его лучше кидать в конец документа.

 

id применял к единичным блокам (те блоки которые встречаются один раз), где это сказано что имя wrapper обязательно должен быть именно классом?, мне всегда казалось что имена id и class выбирает разработчик.

Что означает избыточность css?.

Давно верстал уже не помню по какой причине применил jquery 1,7 а не более новую, но это мне не мешает ведь js он и в Африке js  .  

Вот еще и эти оцените если не трудно, буду весьма признателен за критику.

 

http://yuriy1983.ucoz.ua/doc/site_1

 

http://yuriy1983.ucoz.ua/doc/site_2

 

http://yuriy1983.ucoz.ua/doc/site_3

Edited August 7, 2014 by Raura

[Galaxy]

Хорошим тоном считается использовать id только для js, в верстке нужно обходится классами. 

[Homa]

Хорошим тоном считается использовать id только для js, в верстке нужно обходится классами. 

 

Тоесть, если у меня блок "корзина" с id, на который навешена верстка это тоже плохой тон?

[Raura]

Хорошим тоном считается использовать id только для js, в верстке нужно обходится классами. 

 

Ясно, спасибо! 

[Galaxy]

 

Хорошим тоном считается использовать id только для js, в верстке нужно обходится классами. 

 

Тоесть, если у меня блок "корзина" с id, на который навешена верстка это тоже плохой тон?

 

На id лучше не навешивать верстку, обходитесь классами.