Jump to content

Прошу критики моей первой верстки


dpx
 Share

Recommended Posts

Здравствуйте, жители форума!

Это первый шаблон, который я сверстал - bit.ly/TGJrob. Прошу конструктивной критики моей вёрстки. Интересует HTML-документ на соответствие HTML5, а также критика по CSS-коду. Попиксельного совпадения не добивался.

В процессе вёрстки делал типичную ошибку:

<div class="line-bg"></div>

Но потом исправился и нашёл, кмк, более адекватный вариант.

Шаблон уже выкладывался на этом форуме, но в код я не смотрел, полностью делал всё сам. Тем более, что там были некоторые неточности при вёрстке.

Post Scriptum:

Каждого, кто оставит конструктивное сообщение отблагодарю плюсом в репутации (как только наберу необходимое количество сообщений), сейчас у меня "исчерпан лимит".

Link to comment
Share on other sites

1. Если сжать окно до ? 800px по ширине, то хедер и футер ведут себя некорректно. А именно фон у них.

2. header вроде как используется для «шапки» сайта, а использование его в статье вроде как и не уместно. Могу и ошибаться.

3. В footer-right зачем-то вложен p. Зачем?

А так вроде ништяк — мне нравится.

  • Like 1
Link to comment
Share on other sites

1. Если сжать окно до ? 800px по ширине, то хедер и футер ведут себя некорректно. А именно фон у них.

2. header вроде как используется для «шапки» сайта, а использование его в статье вроде как и не уместно. Могу и ошибаться.

3. В footer-right зачем-то вложен p. Зачем?

А так вроде ништяк — мне нравится.

1. Можно, пожалуйста, скриншот. Уже нашёл в чём причина, по неопытности не знал, как тестировать. Надо будет разобраться. Видимо min-width нужно указать?

UPD: Да, min-width: 940px всё исправил.

2. Недавно читал статью: Избегаем популярных ошибок в HTML5. Там рассказывали про неправильное использование <hgroup> и приводили вот такой пример:

	<header>
<h2>Perfect Logic</h2>
<p>All you want your website to do.</p>
</header>

И просматривая исходный код разных гуру я видел даже неправильное, кмк, использование <header>:

	<article>
<header>
<h2>Perfect Logic</h2>
</header>
<p>Sed ut perspiciatis unde omnis iste natus error.</p>
</article>

Насколько я понимаю, был также вариант использовать <hgroup>, но мне такая конструкция показалась излишней.

	
<article>
<hgroup>
<h1>Perfect Logic</h1>
<h2>All you want your website to do.</h2>
</hgroup>
<p>Sed ut perspiciatis unde omnis iste natus error.</p>
<article>

3. Ну, я честно говоря на автомате написал <p>, но я не знаю насколько это можно расценивать как ошибку. Всё равно же, если там добавится новая строка - нужно будет указывать <p>?

Спасибо за отзыв.

Ещё хотел спросить: этот макет натянуть на какую-то CMS возможно или с этой точки зрения он слишком корявый?

Edited by dpx
Link to comment
Share on other sites

Я немного по html5, h1 отсутствует, структура начинается с Lorem ipsum

http://i.imgur.com/RW5wz.png иерархию надо соблюдать.

article>header>p можно поменять на small

В целом хорошо.

Где написано что h1 должен обязательно присутствовать?

Я вот изучаю и <hn> не использую(

ну не нравится он мне <_<

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy