Оцените качесво вёрстки

[hristofor]

http://errivol.hol.es/

 

Буду очень рад выслушать ваши замечания , советы , упрёки , и конечно же вашу критику.

[Galaxy]

особо не вдавался  но бэк шапки нужно было делать через градиент + картинки, вместо id лучше использовать классы, футер также лучше было сделать css-ом а не картинкой.

[Ludwig Voltman]

Я недавно начал изучать css, поэтому где-то могу ошибаться, но вот как бы я сверстал:

1) название сайта я сделал бы заголовком h1

2) стрелочки у ссылок в блоке навигации нужно делать через свойство background, а не через img. Img только для ключевых изображений (фотки, логотип сайта и т.п.)

3) в верхнем навигационном меню (где ссылки home) очень сложная разметка (так ли нужен тег span, если стили можно задать и через nav, и через ul, и через li, и через a?)

4) очень длинный и ненужный каскад в css (лапша), например: .ff ul li a span. Небольшое изменение структуры меню и код придется переписывать. И как-то span, вложенный в a непривычно выглядит.

5) оформление параграфа с классом text1 мало отличается от оформления обычного параграфа. Зачем создавать лишний стиль ради margin? ведь margin можно задать и другим элементам (заголовку). text-align: justify, если не ошибаюсь, работает только в том случае, если строка заполнена целиком, поэтому смысла в классе text1, имхо, нет

6) содержимое тега blockquote должно быть вложено в тег p (в html4 strict точно, хотя, возможно для html5 это валидно)

7) в форме contact - зачем столько ul? 0_o

8) зачем создавать отдельный стиль для h3 только ради зеленого подчеркивания в правом блоке, если можно использовать два стиля (обычный для оформления h3 и стиль underline {border-bottom: 1px solid green;} Это касается и остальных элементов с подчеркиванием.

9) если не ошибаюсь, свойства класса wrapper можно передать body, так мы избавимся от лишнего дива в html.

10) по оформлению: лучше чтобы была какая-то система наименования классов и идентификаторов. Пока что вперемешку используются символы нижнего подчеркивания вперемешку с CamelCase, тяжело будет разбираться в таком коде.

11) в таблице, вместо того, чтобы задавать классы каждому столбцу, легче задать класс только четным/нечетным, и добавить оформление для тега tr.

Edited November 26, 2013 by Ludwig Voltman

[hristofor]

Я недавно начал изучать css, поэтому где-то могу ошибаться, но вот как бы я сверстал:

1) название сайта я сделал бы заголовком h1

2) стрелочки у ссылок в блоке навигации нужно делать через свойство background, а не через img. Img только для ключевых изображений (фотки, логотип сайта и т.п.)

3) в верхнем навигационном меню (где ссылки home) очень сложная разметка (так ли нужен тег span, если стили можно задать и через nav, и через ul, и через li, и через a?)

4) очень длинный и ненужный каскад в css (лапша), например: .ff ul li a span. Небольшое изменение структуры меню и код придется переписывать. И как-то span, вложенный в a непривычно выглядит.

5) оформление параграфа с классом text1 мало отличается от оформления обычного параграфа. Зачем создавать лишний стиль ради margin? ведь margin можно задать и другим элементам (заголовку). text-align: justify, если не ошибаюсь, работает только в том случае, если строка заполнена целиком, поэтому смысла в классе text1, имхо, нет

6) содержимое тега blockquote должно быть вложено в тег p (в html4 strict точно, хотя, возможно для html5 это валидно)

7) в форме contact - зачем столько ul? 0_o

8) зачем создавать отдельный стиль для h3 только ради зеленого подчеркивания в правом блоке, если можно использовать два стиля (обычный для оформления h3 и стиль underline {border-bottom: 1px solid green;} Это касается и остальных элементов с подчеркиванием.

9) если не ошибаюсь, свойства класса wrapper можно передать body, так мы избавимся от лишнего дива в html.

10) по оформлению: лучше чтобы была какая-то система наименования классов и идентификаторов. Пока что вперемешку используются символы нижнего подчеркивания вперемешку с CamelCase, тяжело будет разбираться в таком коде.

11) в таблице, вместо того, чтобы задавать классы каждому столбцу, легче задать класс только четным/нечетным, и добавить оформление для тега tr.

 

Не подскажешь , нужно ли для стрелки (2) создавать отдельный тег (span) , или стрелку нужно создавать в теге a???

 

Edited November 26, 2013 by hristofor

[roman_kr_ru]

#container {

    height: 1400px;

}

Для чего? Если добавить контент, например в .under, таблица и форма уедут вниз.

 

 

 

или стрелку нужно создавать в теге a

Можно в а создать бэкграундом, я думаю.  

Edited November 26, 2013 by roman_kr_ru

[swandev]

http://errivol.hol.es/

 

Буду очень рад выслушать ваши замечания , советы , упрёки , и конечно же вашу критику.

Не используйте ID для определения стилей.

[Ludwig Voltman]

Не подскажешь , нужно ли для стрелки (2) создавать отдельный тег (span) , или стрелку нужно создавать в теге a???

 

Там и без дополнительного span есть несколько вариантов. background можно присвоить либо тегу li, либо тегу a. Я бы присвоил li.