0 Всего найдено: 10

serega12 Сообщение 20/01/2011 16:22 Копия темы
Код сайта Оцените пожалуйста код, буду очень благодарен, + в карму

egor_fedorov Сообщение 20/01/2011 16:25 Копия темы
норм, только мне кажется

style='float:right; margin-right:200px;'

ставить " " :) привычка )

DErekQ Сообщение 20/01/2011 16:28 Копия темы
Не ровный код, мало комментариев, много использованного кода, я думаю могло быть меньше.
Стили написаны прямо в НТМЛ
к примеру
style='width:1px; height:1px; clear:both'
это не приемлемо, есть для этого CSS.
Да + еще не хватает мета тегов в <head> </
Остольное надо смотреть в полной верстке и сверять с дизайном + так же во всех популярных браузерах.

portfoliant Сообщение 20/01/2011 16:36 Копия темы
стили вынеси в css а так вполне норм

proit Сообщение 20/01/2011 17:06 Копия темы
кодировка не правильная

Xap4o Сообщение 20/01/2011 17:17 Копия темы
причем тут кодировка???
это вообще не косяк в верстке.
щас хостинги поддерживают как utf8 так и 1251, без разницы.

Xap4o Сообщение 20/01/2011 17:20 Копия темы
структура впринципе нормальная, код грязный.
еще, извините если это не играет роли, то зачем стили писать с заглавных букв???

unart Сообщение 20/01/2011 17:21 Копия темы
1) в xhtml имена классов и инлайновые стили принято открывать и закрывать двойной кавычной, а не одинарной. хотя и то и то валидно.
2) инлайновые стили – уныло. надо выносить в css. инлайновые стили допустимы при модификации объекта через javascript
3) javascript по-хорошему надо вынеси в конец кода, т.е перед </body> даже по той простой причине, что до полного формирования DOM он работать не будет.
4) непонятно длинные вложенности элементов, 0 элементов списков, 0 элементов заголовков, слабоуловимая семантичность кода, отсутствие элементарного типографирования "рыбного" текста.
5) отсутствие пары обязательных атрибутов alt у картинок (ну это не принципиально, но сам факт) + атрибут href у div (о_О). метод передачи данных формы пишется маленькими буквами, а не POST
6) использование font-style:italic не кошерно, так как для этого есть специальный тег <i>.

честно говоря я бы такую верстку не принял.

proit Сообщение 20/01/2011 17:51 Копия темы
написано utf8, а на самом деле 1251

redpixel Сообщение 20/01/2011 19:52 Копия темы
К вышесказанному:
вместо
<img class='star' src='img/star1.gif' alt=''/>
педантичнее
<img class='star' src='img/star1.gif' alt='' />

с пробелом перед /> И лучше, без пустых alt=''. Или alt='что-то внятное', или без alt вообще. Соглашусь с предыдущим мнением, все стили только в CSS.
Удачи !