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

[pamparam]

Доброго времени суток!!!!
 

Я начинающий ,  для меня важна критика ...

сверстал страницу по этому шаблону http://htmlforum.ru/index.php?showtopic=13276
вот что получилось http://priviev.bl.ee/

 

спасибо за комментарии

[wendigo]

1) Ужасные имена классов(adres, lupa) + нижние подчеркивания(это не php, в css предполагается использовать дефис в качестве разделителя).

2) Использование inline стиля ничего хорошего в себе не несет.

<div style="margin: 40px auto 30px;">

3) Футер не выровнен как на макете, должен быть правее.

4) Повторяющийся код, пример:

.footer_left p {margin: 10px 0;padding: 0;}.footer_right p {margin: 10px 0;padding: 0;}

5)  Во многих местах margin можно сократить

#block_left div {display: block;margin-left: 25px;font-size: 13pt;;margin-top: 20px;text-align: left;margin-bottom: 25px;}

7) Отступ между классами. Где-то он есть, где-то его нету. Где-то вообще лишние строки. Неаккуратно написан css.

8) Зачем нужен этот комментарий?

bg_rgb_logo {/** background: url('img/rgb_logo.png') repeat-x; */

9) Группированные селекторы лучше размещать на разных строчках.

Edited February 16, 2014 by wendigo

[js5]

Для очистки потока документа используй псевдокласс ::after для добавления блочного псевдоэлемента. Вместо 

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

[CroaToa]

Доброго времени суток!!!!

 

Я начинающий ,  для меня важна критика ...

сверстал страницу по этому шаблону http://htmlforum.ru/index.php?showtopic=13276

вот что получилось http://priviev.bl.ee/

 

спасибо за комментарии

А у меня футер вообще выравнен к левому краю, и при разрешении 800*600 заканчивается под началом "Наш адрес". Короче не идет до конца) Картинки Росгосстрах, асто и согласие надо было делать отдельно, кликабельные. Текст наверху "Круглосуточная консультация" Не по макету. (все это на разрешении 800*600).

 

Использование #id вместо .class много раз обсуждалось на форуме. Дайте возможность программистам работать после Вас с сайтом.

 

Не правильно div class="adres", Правильно div class="address" - Делайте привычку написать слова правильно, заодно и англ научитесь. 

 

Комментарии на русском ужасны.

 

Footer left и footer right, если честно смысл этого не понял, зачем создавать 2 блока (я тоже начинающий, может еще не понимаю, если что обясните).

[pamparam]

1) Ужасные имена классов(adres, lupa) + нижние подчеркивания(это не php, в css предполагается использовать дефис в качестве разделителя).

2) Использование inline стиля ничего хорошего в себе не несет.

<div style="margin: 40px auto 30px;">

3) Футер не выровнен как на макете, должен быть правее.

4) Повторяющийся код, пример:

.footer_left p {margin: 10px 0;padding: 0;}.footer_right p {margin: 10px 0;padding: 0;}

5)  Во многих местах margin можно сократить

#block_left div {display: block;margin-left: 25px;font-size: 13pt;;margin-top: 20px;text-align: left;margin-bottom: 25px;}

7) Отступ между классами. Где-то он есть, где-то его нету. Где-то вообще лишние строки. Неаккуратно написан css.

8) Зачем нужен этот комментарий?

bg_rgb_logo {/** background: url('img/rgb_logo.png') repeat-x; */

9) Группированные селекторы лучше размещать на разных строчках.

Спасибо ...

со всем  пуду работать ....

 

 

Для очистки потока документа используй псевдокласс ::after для добавления блочного псевдоэлемента. Вместо 

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

Спасибо

не использовал так как не потдерживается ИЕ7

 

 

 

Использование #id вместо .class много раз обсуждалось на форуме. Дайте возможность программистам работать после Вас с сайтом. Не правильно div class="adres", Правильно div class="address" - Делайте привычку написать слова правильно, заодно и англ научитесь. Комментарии на русском ужасны. Footer left и footer right, если честно смысл этого не понял, зачем создавать 2 блока (я тоже начинающий, может еще не понимаю, если что обясните).

 

Спасибо,

да с англ. у меня проблемы

использовал два - так показалось легче

Edited February 17, 2014 by pamparam