Оцените верстку(Wordpress не трогайте)

[psixodyb]

Собственно сам сайт.

http://bananabet.ru/ 

[Laroxx12]

Картинки можно было сделать спрайтами..

А так мне нравится

[Vlad02]

Два доктайпа это фича?

И в css сплошные id и теги.

Edited October 5, 2014 by Vlad02

[psixodyb]

Два доктайпа это фича?

Исправлено, немного криво пихнул шаблон в wordpress

[klierik]

1. Стили написаны плохо, все зашито под #id и не оптимизировано. Незачет

2. 

header nav ul {width: 480px;padding: 0px;margin: 0px auto;}

Почему меню имеет жесткую ширину? А что будет елси завтра заказчик добавит пару пунктов?

 

3. На главной только странице куча <H1>, хотя должен быть только один.

 

4. это

<div class="table">				<p class="top-table date">Дата</p><p class="top-table sobitie">Событие</p><p class="top-table bet">Ставка</p>				<p class="top-table koef">Коэф.</p><p class="top-table yesno">Исход</p>				<div class="content-table">				<p><span>31 августа</span><span> Реал Сосьедад – Реал М; Локомотив – Зенит</span><span>ТБ2.5; П2</span><span> .2.51</span><span> Сыграл</span></p><div id="wpcr_respond_1"></div>				 				 </div>				</div>

наверно один из самых странных кусков кода который я когда либо видел. ну вот зачем делать таблицу из параграфов и дивов? чем тебе <table> не угодил для табличных-то данных?

 

Это не верстка, это просто набор тегов, о предназначении который автор не знает, описанный стилями, форматировать которые он не умеет.

Но, по крайней мере у ТС получилось это дело хоть как-то собрать, что уже хорошо.

 

Желаю набираться опыта

 

PS: в подписи ссылка на типичные ошибки, рекомендую ознакомится

[psixodyb]

1. Стили написаны плохо, все зашито под #id и не оптимизировано. Незачет

2. 

header nav ul {width: 480px;padding: 0px;margin: 0px auto;}

Почему меню имеет жесткую ширину? А что будет елси завтра заказчик добавит пару пунктов?

 

3. На главной только странице куча <H1>, хотя должен быть только один.

 

4. это

<div class="table">				<p class="top-table date">Дата</p><p class="top-table sobitie">Событие</p><p class="top-table bet">Ставка</p>				<p class="top-table koef">Коэф.</p><p class="top-table yesno">Исход</p>				<div class="content-table">				<p><span>31 августа</span><span> Реал Сосьедад – Реал М; Локомотив – Зенит</span><span>ТБ2.5; П2</span><span> .2.51</span><span> Сыграл</span></p><div id="wpcr_respond_1"></div>				 				 </div>				</div>

наверно один из самых странных кусков кода который я когда либо видел. ну вот зачем делать таблицу из параграфов и дивов? чем тебе <table> не угодил для табличных-то данных?

 

Это не верстка, это просто набор тегов, о предназначении который автор не знает, описанный стилями, форматировать которые он не умеет.

Но, по крайней мере у ТС получилось это дело хоть как-то собрать, что уже хорошо.

 

Желаю набираться опыта

 

PS: в подписи ссылка на типичные ошибки, рекомендую ознакомится

Понимаю что табличные данные нужно делать в таблице. Но я сделал так для более простого(на мой взгляд) добавления данных из самого вордпресса. У меня не пришло больше никаких идей как сделать легкое добавление данных в cms и красивую верстку. Хотя самого воротит от этих параграфов=)))

Edited October 5, 2014 by psixodyb