Оцените, please, вёрстку

[black40]

Оцените вёрстку страницы сайта, пожалуйста... http://building.esy.es/

Интересно Ваше мнение о css-файле.

Да, и не обращайте внимание на количество рекламы Ну дизайн такой(не мой , конечно)

 

    Спасибо.

[CroaToa]

а почему только мнение о css? в html то много проблем, валидатор ругается со всей силы. Категории не ссылки, в css отступы не урегулированы, не красиво смотрится. 

 

зачем вот этот код? 

<!--[if lt IE 9]>   <script>      document.createElement('header');      document.createElement('nav');      document.createElement('section');      document.createElement('article');      document.createElement('aside');      document.createElement('footer');   </script><![endif]-->

[afdw]

зачем вот этот код? 

<!--[if lt IE 9]>   <script>      document.createElement('header');      document.createElement('nav');      document.createElement('section');      document.createElement('article');      document.createElement('aside');      document.createElement('footer');   </script><![endif]--> 

http://htmlbook.ru/samlayout/verstka-na-html5/novye-tegi

Edited May 28, 2014 by afdw

[black40]

зачем вот этот код? 

Для IE 8. Иначе он не понимает стилей для header, nav ну и т.д. 

 

зачем вот этот код? 

<!--[if lt IE 9]>   <script>      document.createElement('header');      document.createElement('nav');      document.createElement('section');      document.createElement('article');      document.createElement('aside');      document.createElement('footer');   </script><![endif]--> 

http://htmlbook.ru/samlayout/verstka-na-html5/novye-tegi

 

Документ не найден

[CroaToa]

Для IE 8. Иначе он не понимает стилей для header, nav ну и т.д.

  Я не очень понимаю, если это для IE8 то почему начинать скрипт с if lt IE 9 ?

[afdw]

 

Для IE 8. Иначе он не понимает стилей для header, nav ну и т.д.

  Я не очень понимаю, если это для IE8 то почему начинать скрипт с if lt IE 9 ?

 

http://htmlbook.ru/samlayout/internet-explorer/uslovnye-kommentarii

зачем вот этот код? 

<!--[if lt IE 9]>   <script>      document.createElement('header');      document.createElement('nav');      document.createElement('section');      document.createElement('article');      document.createElement('aside');      document.createElement('footer');   </script><![endif]--> 

http://htmlbook.ru/samlayout/verstka-na-html5/novye-tegi

 

Документ не найден

В конце ссылки добавился пробел 

Edited May 28, 2014 by afdw

[black40]

в html то много проблем, валидатор ругается со всей силы.

А по моему не особо... alt не проставлен - так это в банерах(они скопированы), в section нет заголовков - по дизайну их там нету ну и формат даты-времени, он так по дизайну проставлен. 

Я не очень понимаю, если это для IE8 то почему начинать скрипт с if lt IE 9 ?

 

Я так понимаю это значит, что скрипт работает для IE до 9 версии 

Пример 6.4. Скрипт для IE

<!--[if lt IE 9]>
<script>
var e = ("article,aside,figcaption,figure,footer,header,hgroup,nav,section,time").split(',');
for (var i = 0; i < e.length; i++) {
document.createElement(e);
}
</script>
<![endif]-->

Сам скрипт заключается в условные комментарии, чтобы выполнялся только для IE версии 8.0 и ниже. В IE9 поддержка новых тегов HTML5 уже включена.

[Shenia132]

сайт не влазит в разрешение 1024px а это плохо 

[black40]

сайт не влазит в разрешение 1024px а это плохо 

Да, не влазит. В PSD стоит ширина 1200рх. И ТЗ нету.(Откуда ж ему взяться то?) Так и поставил - 1200рх. При "резине" это, конечно, надо учитывать.

А файл стилей смотрит кто-нибудь? Валидатор весь красный от гнева А как исправить ошибки - я хз(

 

Ну если только - не обращать внимания на IE?

Edited May 28, 2014 by black40

[klierik]

в данной верстке смотреть нечего так как толком верстки нет.

ссылка на типичные ошибки -- в подписи

[Shenia132]

 

сайт не влазит в разрешение 1024px а это плохо 

Да, не влазит. В PSD стоит ширина 1200рх. И ТЗ нету.(Откуда ж ему взяться то?) Так и поставил - 1200рх. При "резине" это, конечно, надо учитывать.

А файл стилей смотрит кто-нибудь? Валидатор весь красный от гнева А как исправить ошибки - я хз(

 

Ну если только - не обращать внимания на IE?

 

поставил бы min-width:1000px и max-width:1200px

Ошибки,если плохо с английским то переводи translate.google.ru я так и делаю, хотя там и так понятно все

[black40]

ссылка на типичные ошибки -- в подписи

Смотрел и раньше. А можно конкретней по ошибкам?

в данной верстке смотреть нечего так как толком верстки нет.

Страница браузерами выводится...

Что я делаю не так? 

[klierik]

Страница браузерами выводится... Что я делаю не так?

я имею ввиду что страница настолько простая что в целом ее объем не велик, и оценивать сложно, так как нечего толком оценивать.

Тут же несколько блоков и лейаут. Сделай что-то покрупнее, каталог, например, веб-магазин или портал.

из ошибок:

1. тег <p> использован там где не надо. <p> -- используется для абзацев, а не для того что бы вывести пару слов.

2. Блок "О Компании" на практике будет иметь совершенно другую разметку. Дело в том что контент менеджер не будет заморачиватся тем что бы добавлять к каждому параграфу и заголовку классы. Этот контент будет создан редактором. В качестве примера редактора могу рекомендовать: http://www.tinymce.com/

[black40]

я имею ввиду что страница настолько простая что в целом ее объем не велик, и оценивать сложно,

Фух, полегчало ... А то уже подумал - "наверстал", блин. А вот на счёт портала - магазина, не рановато мне? Вот такие вот, помельче, научиться бы верстать для начала. 

 

 

Блок "О Компании" на практике будет иметь совершенно другую разметку.

Как же его тогда надо было верстать?

Edited May 29, 2014 by black40

[black40]

Сделай что-то покрупнее, каталог, например, веб-магазин или портал.

А хотя....PSD не подкинете, с ТЗ. Только чтоб по функционалу не слишком сложно, а то застряну надолго) 

[klierik]

http://htmlforum.ru/index.php?showforum=31

Сюда я выкладывал 3 задания с ТЗ -- выбирай любой