Jump to content

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


Heidel
 Share

Recommended Posts

1. отсутствие hover, active

2. section != wrap/container и не служит для построения секций, как можно было бы предположит из названия

3. зачем title на img?

4. article в aside? не логично. Если это aside, то в нем не может находиться основной контент страницы.

5. навигация в футере - не логично, ИМХО.

6. старайся обходиться без <div class="clear"></div>

П.С. скорость слайдера раздражает...) Стоило бы увеличить как мин в 2 раза, ИМХО.

Edited by Viper
Link to comment
Share on other sites

article в aside? не логично.

По-моему нормально. Разве не может быть самодостаточных (для агрегаторов и т.п.) единиц информации, относящихся к основному контенту страницы лишь касательно?

Вот насчет nav в футере — согласен, что-то одно из них там, скорее всего, лишнее.

  • Like 1
Link to comment
Share on other sites

Разве не может быть самодостаточных (для агрегаторов и т.п.) единиц информации, относящихся к основному контенту страницы лишь касательно?

Почему же, может. И такие единицы информации, лично я, отношу к aside.

Если задуматься, то наверное соглашусь с вами данном случае, взглянув на это с другой точки зрения. Статьи относящиеся к основному контенту страницы лишь касательно... что-то в этом есть)

П.С.Пасиб за расширение кругозора)

Link to comment
Share on other sites

article — необязательно статья, он ближе к англ. выражению «article of clothing» :)

Впрочем, в спецификации W3C, которую позавчера волшебным пинком толкнули в статус кандидата в рекомендации, главная фишка этих модных элементов — алгоритм построения «document outline» — оказалась под риском удаления, и многие корифеи высказываются в таком ключе, что неплохо бы и сами элементы туда же... так что, полагаю, не стоит придавать этим мелочам слишком уж много значения :)

Link to comment
Share on other sites

hover и active к сожалению, не были в макете прорисованы, а я, к сожалению, не настолько хорошо владею фотошопом, чтобы самой что-то нарисовать.

title на img все равно если они пустые, то не отображаются.

article в aside класть можно, в книге Кристофер Шмитт, Кайл Симпсон - "HTML5. Рецепты программирования" сказано:

Можно рассматривать article как особую форму section, которая предна­значена для контента, самостоятельно существующего за пределами окру­жающей информации. Например, это сообщения, «объединенные» в блог.

Элемент article можно использовать и для другого контента, в том числе:

- видео и сопутствующих записей;

- новостных статей;

- комментариев к блогу.

Скорость слайдера сделано такой, какая она нравится лично мне, в любом случает, это только учебная верстка и поправить пару цифр можно в любой момент без проблем)

Edited by Heidel
Link to comment
Share on other sites

pincher2012, Не совсем поняла.

То есть, для блока, в котором у меня лежат блоки с позиционированием float: right или float: left, надо задать overflow: hidden; ?

То есть, так -

	<style type="text/css">
#wrap {width: 500px; background: red; overflow: hidden }
#wrap div {width:50px; height: 50px; float: left; background: green; margin:15px;}
</style>

<div id="wrap" >
<div></div>
<div></div>
<div></div>
<div></div>
<div>

?

Edited by Heidel
Link to comment
Share on other sites

То есть, для блока, в котором у меня лежат блоки с позиционированием float: right или float: left, надо задать overflow: hidden; ?

Да, либо использовать clearfix. Вот, нашел статью на русском.

Сравнение различных методов.

Edited by pincher2012
  • Like 1
Link to comment
Share on other sites

И еще одно, с наглядным примером

[offtop]

Что-то Макс там в статье так и не выправил финальный код css...

Я ему присылал лечилку одного багла, которое получалось если применяется нестандартный шрифт,

так же в статье остался код который приводил к перекосу в Opera 12.01+


.clearfix {
-height: 1%;
*zoom: 1;
display: table;
}
.clearfix:after {
content: '. . . . . . . . . . .';
font: .13em/0 sans-serif;
display: block;
word-spacing: 99in;
height: 0;
overflow: hidden;
}

[/offtop]

  • Like 1
Link to comment
Share on other sites

Про Оперу же отдельная большая «песня» была.

это да, но там еще остался в этом месте


/* Лекарство бага с отступом в Opera */
font-size: 0.05em;
line-height: 0.05em;

глюк с сторонними шрифтами, который вылазил, кажется на всех браузерах, по прежнему там, а код выше перестал работать после выхода Opera 12.10

Так в следствии и появилась вместо тех строк, одна такая


font: .13em/0 sans-serif;

Link to comment
Share on other sites

Если Вы используете aside и article то очевидно Вы хоите писать на 5-ке, зачем тогда доктайп такой длинный? Еще кодировка (1251), зачем? у меня к примеру Ваш текст не читабельный (кракозябры горят). Я бы исправил : <!DOCTYPE html>, <meta charset="UTF-8">, еще ошибка meta content="text/html; charset=windows-1251" http-equiv="Content-Type", скобки не закрыли после "content"

Edited by Pawel.rsk
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