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

[oleg126]

Прошу оценить верстку, указать на ошибки ну и помочь кто чем может!)

http://rghost.ru/51383233

[li4e]

Скачивать очень неудобно, используйте бесплатные хостинги для размещения тестовых сайтов!

[oleg126]

Скачивать очень неудобно, используйте бесплатные хостинги для размещения тестовых сайтов!

 

может посоветуете какой бесплатный хостинг, а то их много даже не знаю куда зайти!)

[li4e]

Сам не пользуюсь бесплатными, но вот первый попавшийся в google http://hphosting.ru/freehosting

[oleg126]

Ну вроде все!)

http://oleg126test.freehphosting.ru/site1/

Не очень сложный шаблон конечно, но хочется узнать может уже какие-то ошибки есть?

Edited January 4, 2014 by oleg126

[Aleksandr.L]

А что тут оценивать?

 

P.S. DOCTYPE HTML5, а где семантические теги?

[Vlad02]

Используйте классы вместо id.

 

Если задали #wrapper width:960px то, для чего  #header и #footer опять задаете width:960px? Если считаете правильным задавать width для  #header  и #footer, то почему не задаете width для  #content?

 

height в #wrapper есть ЗЛО...

 

подвал это тихий писец)

 

Разбирайтесь с основами блочной модели.

Edited January 4, 2014 by Vlad02

[oleg126]

А что тут оценивать?

 

P.S. DOCTYPE HTML5, а где семантические теги?

 

Ну правильность кода, все ли нормально сделано, по макету ли, может что то можно было бы не писать или сократить а что то написать и т.д.!

 

И что такое семантические теги? Может я не правильно понимаю, но они же есть у меня header, footer, wrapper, content!

 

Или это не то?

Edited January 4, 2014 by oleg126

[Vlad02]

И что такое семантические теги? Может я не правильно понимаю, но они же есть у меня header, footer, wrapper, content! Или это не то?

 

Нет, не то. В чем различие тега от id?

Должно быть

<header></header><nav></nav>

у тебя везде id

[oleg126]

честно говоря не очень понятно что вы хотите мне донести

[Vlad02]

Есть теги которые были добавлены в HTML 5 ,смотрите здесь

лучше здесь

Edited January 4, 2014 by Vlad02

[oleg126]

2 сайт

http://oleg126test.freehphosting.ru/site2/

пытался исправить замечания, но я так думаю все же недостатков осталось много, если не трудно, можно их показывать именно в коде?

Макет

Edited January 4, 2014 by oleg126

[y6uBaToP]

Данный макет видимо должен быть на всю ширину страницы, а не на 1600px. #wrapper {width:100%}

Странное название центральной части сайта header_logo

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

Топ нав не реагирует на hover. Нижнее реагирует.

[oleg126]

Данный макет видимо должен быть на всю ширину страницы, а не на 1600px. #wrapper {width:100%}

Странное название центральной части сайта header_logo

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

Топ нав не реагирует на hover. Нижнее реагирует.

 

Да я уже и сам про это подумал, иначе на мониторах с разрешением менее 1600px выглядит не красиво!

Не понимаю при чем тут название?) По большому счету без разницы что там написано) Мне интересно мнение по семантическим тегам, правильно ли я их использовал и везде ли они у меня стоят?

По поводу картинки, просто не знал как в css сделать такой задний фон со своими свойствами, поэтому решил что итак на сайте всего 4 картинки и это не слишком загрузит сайт!

Ну а про топ нав это дело моей забывчивости всего лишь)

 

Но все равно спасибо за замечания!