Jump to content

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


p1O
 Share

Recommended Posts

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

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

Edited by red4pony
Link to comment
Share on other sites

Для создания горизонтальных разделителей лучше использовать <hr>

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

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

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

Link to comment
Share on other sites

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

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

Нет логического разбиения элементов по блокам (header, main)

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

Все в одной оболочке, не очень удобно для глаз.

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

- для переноса текста, я не для создания отступов (посмотрите эту тему).

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

<center> замените на <div> или <span>

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

Link to comment
Share on other sites

В исходном коде <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>
<!--Ниже как правило ничего не размещают-->

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

Link to comment
Share on other sites

В исходном коде <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?

Link to comment
Share on other sites

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

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

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