бэкграунд для кнопок лучше делать css-спрайтами , а не тегом <img>.

Валидатор в помощь.Т.к. присутствуют ошибки в CSS,HTML.А так по сути и оценить больше нечего.На сайте всего одна страница,на которой толком ничего нет.

на самом деле по одной странице можно уже получить представление об уровне вёрстки.
В данном сайте уже очевидны устаревшие решения
1.Doctype
2. Верстка сайта Table а не Div
3. Использование height для блоков с контентом, т.к. контент может увеличиться. то это приведёт к багу.
4. Нет clearfix подстраховки для IE. так как применяются float и еслиб не высота блоков, то возможно перекосило бы в Експлоере.

Можно и больше сказать если присмотреться.....

Спасибо, ошибку с отсутствием тега tr исправил. Невнимателльность :rolleyes:
Теперь HTML валиден. А в CSS невалидность из-за хаков под ослика. Пусть остается, ведь поисковики все равно не смотрят на CSS, а значит валидность CSS файла не так уж и критична.



Оценивать качество верстки. Это не сайт, а всего лишь веб-страница, сверстанная мной по макету http://forum.htmlbook.ru/index.php?showtopic=29381 в учебных целях. К слову, моя первая блочная верстка. А сам сайт вот он http://www.kinoruletka.ru/ и я не являюсь ни его автором, ни владельцем.



Уровень верстки у меня чуть выше начального, но я стремлюсь к лучшему.

1. Doctype. В данном случае сайт верстался под разные браузеры, в том числе и IE старых версий, которые не поддерживают HTML5 и CSS3. Поэтому пришлось ограничиться HTML4 и CSS2. Doctype строгий, и я не вижу причин, по которым это может быть неправильно.

2. Верстка полностью делалась на divaх, за исключением каркаса страницы. Можно было сделать и на divax тоже, но я решил применить таблицу для разнообразия. Таким образом, в коде всего один раз встречается тег table. Называть табличную верстку устаревшей я бы не стал. На мой взгляд не стоит зацикливаться ни на блочной верстке, ни на табличной, применяя и то и другое исходя из целесообразности в каждом конкретном случае. Об этом, кст, где-то на htmlbook.ru есть статья.

3. Контент блоков, исходя из сути сайта, меняться не будет. Именно поэтому все версталось жестко. В том числе и высота.

4. Под IE использовались хаки. И я по окончании верстки проверял работоспособность страницы в 6 основных браузерах. Если бы что-то пошло не так, я бы искал решения.

А вот за спрайты - огромное спасибо, потому что с ними я вообще не был знаком. А ведь это, как я посмотрю, реально мощная техника! Пока img на background в CSS менять не стал, потому что у меня нет совемщенных изображений для придания динамичности кнопкам. Но ради тренировки, сделаю кнопки divaми со статичными backgroundами, отказавшись от img.

Если можете сказать больше, то буду Вам очень признателен. Именно для этого я и выложил сюда работу.

И спасибо за уже изложенную конструктивную критику!

1.Элемент <!DOCTYPE> кроссбраузерность начиная с IE6. Сомневаюсь что поддерживаете IE5.
2. Вы защищаете табличную вёрстку, у вас сай держится на таблице, уберите её и думаю все развалится. Вот это и называется табличная вёрстка, а не количество таблиц в сайте....
я отказываюсь больше что-то говорить.... Парень свято верит в то что сделал. Я сам с так рассуждал, когда через год работы посмотрел на свою первую вёрстку, то ужаснулся ))

Откройте исходный код в мозиле и увидите свою ошибку.Не правильное вложение тэгов.

По поводу всего выше сказаного солгасен с gen-5

По поводу IE, в 6-м у вас криво отображается все.

По поводу хаков,я на своем сайте для верстки не использовал ни каких хаков,и все отлично отображается.Начиная с IE8 ,под 7 не подгонял,хотя это возможно сделать, так же без хаков.В сторону ie6 даже не смотрю.Ставлю заглушку.

П поводу табличной верстки так же считаю, что это не правильно.Хотя даже сейчас встречаются опытные вебмастера, которые считают иначе,что в табличной верстке нет ничего плохого,и она легче осваевается чем div.

P.S. Я недавно начал обучаться,и заниматься разработкой сайтов.Но начал сразу с div,как то давно пробовал табличную верстку.ИМХО мне понятней и легче намного освоились div чем table.Таблицами ни когда не верстал и даже незнаю как на них верстать.)) А вот на div все понятно и легко.

1. На самом деле я и на IE6 не равнялся. Кроссбраузерность при верстке обеспечивалась, начиная с IE7. Т.е. я так понимаю, что, если я напишу <!doctype html>, то это не будет ошибкой, даже если я не использую в коде ни одного элемента из HTML5?

2. Ради тренировки буду сегодня пытаться переделать каркас, чтобы он был на divax. За одно сделаю кнопки divaми c backgroundами.

Мне кажется, все мы должны верить в то, что делаем, но это уже из области философии :rolleyes:




Я понял, почему мы говорим о разных вещах. Ошибки возникают начиная с тегов, которые идут в коде после </body> и </html>. Это типа код рекламы от бесплатного сервера, на котором я разместил страничку. В этом коде - да, сплошь неправильное вложение тегов. Но к моей верстке этот код не имеет никакого отношения. Если проверять на валидность код от <!doctype ...> до </html>, то он валиден! Ошибок нет!

По IE я уже написал выше. Тоже считаю нецелесообразным верстать под IE6, исходя хотя бы даже из таких данных: http://www.liveinternet.ru/stat/vkontakte.ru/browsers.html.

По поводу отношения к табличной верстке говорить больше ничего не буду. Такие споры, насколько я знаю, идут постоянно. Зачем устраивать еще один? :rolleyes:

Вот, готов второй вариант верстки http://kinoruletka.h16.ru/index2.html
Теперь таблицы не используются вообще, а кнопки сделаны без img (т.е. как раз с возможностью применения css-спрайтов).

Ну так вполне нормально.

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

Как пример,макет приведенный ниже: не то что бы он сложный.Но когда я только начинал, для меня было сложно подобное сделать,я даже не знал как такой сделать)).

Можо еще чего ни будь своего добавить сюда.По моему подобную структуру используют многие новостные сайты.