Критика верстки макета

[Diadumen]

Подскажите плиз какие ошибки были допущены при верстке

http://valentgostilian.elitno.net/Site1/index.html

Заранее спасибо

[Viper]

1. Попытайтесь выделить номер телефона и текст - "Согласно Лицензии на оценочную..."

2. Отсутствуют hover у ссылок

3. <br> между тегами - плохо и бессмысленно.

4.

footer section p, .address {... padding-left: 0 !important; ...}

- зачем !important?

5.

.blockNav nav ul li a

- старайтесь избегать длинных каскадов

Edited March 19, 2013 by Viper

[Березовский]

Все отлично!) рябит только от изобилия инфы, текста) а так очень даже прилично выглядит.

[Diadumen]

1. Попытайтесь выделить номер телефона и текст - "Согласно Лицензии на оценочную..."

2. Отсутствуют hover у ссылок

3. <br> между тегами - плохо и бессмысленно.

4.

footer section p, .address {... padding-left: 0 !important; ...}

- зачем !important?

5.

.blockNav nav ul li a

- старайтесь избегать длинных каскадов

1. Пока этот пункт пропущу ткак как в макете этот блок никак не выделен...

2. hover поставил

3. br убрал

4. !important там действительно не нужен

5. .blockNav nav а лучше так? Почену избегать длинных каскадов?

Спасибо за ответ

Edited March 24, 2013 by Diadumen

[cyklop77]

у вас высота подвала фиксированная

Edited March 24, 2013 by cyklop77

[Viper]

5. .blockNav nav а лучше так? Почену избегать длинных каскадов?

Да, так лучше. А в идеале лучше дать ссылкам классы и через них менять стили. Но это не ошибка, а "путь" достижения лучшего результата, который используеться в зависимости от требований. Имеет значения, обычно, для больших нагруженых сайтов.

Почему? Вкратце: чем длинее каскад селекторов, тем дольше будут применяться стили. Подробнее: _http://www.xiper.net/learn/css/efficient-css/efficient-css-selectors.html

Если максимизировать использование классов, то так же упроститься поддержка проекта, ре-дизайн и тд..

1. Пока этот пункт пропущу ткак как в макете этот блок никак не выделен...

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

[Diadumen]

1. Пока этот пункт пропущу ткак как в макете этот блок никак не выделен...

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

Да, сначала не так понял