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

p1O · May 23, 2013

Добрый вечер.

Прошу оценить код и как сверстано на http://break.zg5.ru/1/index.html

Также указать ошибки и на что следует обратить внимание.

Спасибо

red4pony · May 23, 2013

Для создания горизонтальных разделителей лучше использовать <hr>. <br> - для переноса текста, а не для создания отступов (посмотрите эту тему). Нет логического разбиения элементов по блокам (header, main). Все в одной оболочке, не очень удобно для глаз. <center> замените на <div> или <span>

P.S. Первое предложение ошибочно. Поторопился.

Edited May 23, 2013 by red4pony

lexxcode · May 23, 2013

On 5/23/2013 at 3:08 PM, red4pony said:
Для создания горизонтальных разделителей лучше использовать <hr>

у него вроде единственный разделитель, тот который отделяет футер, сделан бордером и это правильно.

Там где заголовок сделано картинками, ну можно было обойтись решением на css.

В остальном согласен

p1O · May 23, 2013

On 5/23/2013 at 3:54 PM, alexriz said:
Там где заголовок сделано картинками, ну можно было обойтись решением на css.

Сделал через CSS

On 5/23/2013 at 3:08 PM, red4pony said:
Нет логического разбиения элементов по блокам (header, main)

Сделал логическое разбиение, но наверно не очень приятное. Если да, то не подскажите как лучше

On 5/23/2013 at 3:08 PM, red4pony said:
Все в одной оболочке, не очень удобно для глаз.

Если про оболочку Wrapper, то ее я убрал или тут я не правильно Вас понял

On 5/23/2013 at 3:08 PM, red4pony said:
- для переноса текста, я не для создания отступов (посмотрите эту тему).

Этот момент я не понял, так как не использовал <br>

On 5/23/2013 at 3:08 PM, red4pony said:
<center> замените на <div> или <span>

Я правильно сделал или ошибся?

red4pony · May 23, 2013

В исходном коде <br> нет, но если посмотреть через Firebug, то перед тегом <center> генерируются два <br>.

<center> - устаревший элемент, замените его на <div> и выровняйте его по центру с помощью CSS (magrin: 0 auto;)

Про разбиение на логические блоки (семантика).


<header>
<h1 class="logo">Мой логотип<h1>
<nav class="menu">
	 <ul></ul>
</nav>
</header>
<section>
<!--тут помещаете основное содержимое сайта-->
</section>
<footer>
<!--Подвал-->
</footer>
<!--Ниже как правило ничего не размещают-->

Имена классов задавайте исходя из предназначения элемента, а не положения на странице(оно может измениться).

p1O · May 23, 2013

On 5/23/2013 at 6:16 PM, red4pony said:
В исходном коде <br> нет, но если посмотреть через Firebug, то перед тегом <center> генерируются два <br>.
<center> - устаревший элемент, замените его на <div> и выровняйте его по центру с помощью CSS (magrin: 0 auto;)
Про разбиение на логические блоки (семантика).
<header>
<h1 class="logo">Мой логотип<h1>
<nav class="menu">
	 <ul></ul>
</nav>
</header>
<section>

</section>
<footer>

</footer>

Имена классов задавайте исходя из предназначения элемента, а не положения на странице(оно может измениться).

Но тега center и в исходном нету, где Вы его нашли о_О

и семантика ведь HTML5, а я делал на HTML 4.01 и тогда что указывать в доктайп 4.01 или уже 5?

red4pony · May 23, 2013

Посмотрите через Firebug или другой инструмент. <center> генерируется вашим хостером и вставляется ссылка внизу страницы. Тут уже никак не изменить.Только приобрести платный аккаунт. (моя невнимательность второй раз за сегодня меня подводит, извините)

Доктайп <!DOCTYPE html>

exeto · May 24, 2013

Любой доктайп отличный от <!DOCTYPE html> давно пора выкинуть на помойку. Вот доклад Вадима Макеева на эту тему.

Sign In

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

Recommended Posts

p1O

Link to comment

Share on other sites

red4pony

Link to comment

Share on other sites

lexxcode

Link to comment

Share on other sites

p1O

Link to comment

Share on other sites

red4pony

Link to comment

Share on other sites

p1O

Link to comment

Share on other sites

red4pony

Link to comment

Share on other sites

exeto

Link to comment

Share on other sites

Join the conversation

Обсуждения

Forums

Important Information