Распространенные ошибки начинающего HTML-верстальщика
Привет, читатель! Я верстаю сайты с 2011 года. Как и большинство верстальщиков того времени, я учился самостоятельно по видео-урокам. За это время, я не раз ошибался с выбором позиционирования блока, выбором тега для элемента и конечно, с названием классов. В этой статье я хочу поделиться своими ошибками.
Я взял один из своих проектов, в котором мы вместе с вами рассмотрим ошибки и исправим их. Все примеры я выложил на GitHub.
Верхняя панель с меню и поиском
Первым блоком на странице является тянущаяся на весь экран верхняя панель, которая содержит меню и поиск. Для начала посмотрим общую структуру блока:
Первым, что бросается в глаза, смотря на нее, это использование элемента div с классом header и элемента header с идентификатором header. Я уверен, что 100% верстальщиков, взглянув на этот код, не поймут, где же действительно шапка. Поэтому, нужно переверстать элементы следующим образом:
Далее, нам требуется определить, для чего используется идентификатор header. Для этого в CSS найдем следующие каскады:
.header{..}
#header{...}
#header .search_block{...}Я не знаю, в каком состоянии я находился, делая CSS-каскады с использованием класса header и идентификатора header, но как говорится, «Из песни слов не выкинешь». Мало того, что в данном случае создание двух CSS-каскадов бессмысленно, так и использование id в CSS лишает вас возможности переиспользовать CSS, а это в свою очередь, ведет к его дублированию. Поэтому, мы будем использовать только класс header. Изменю наш код следующим образом:
.header{..}
.header .search_block{...}Но, честно говоря, у меня большие сомнения, что это шапка сайта. Я больше склоняюсь к тому, что это верхняя панель. Поэтому, советую переименовать класс header в top-panel.
.top-panel{..}
.top-panel .search_block{...}Верстка основного контейнера
Идем дальше по нашей структуре и встречаем div с классом wrapper:
И для полноты картины CSS:
.wrapper{
margin: 0 auto;
width: 940px;
padding: 0 10px;
}Слово wrapper можно перевести как обертка. Не совсем понятно что она оборачивает? Да и этот блок вообще не обертка. Так что с этим классом я погорячился.
Этот элемент служит для центрирования и указания ширины нашего сайта. По сути, это основной блок страницы, в который помещаются все остальные. Поэтому, логично было бы его назвать main-container.
Меню
Теперь детальнее разберем меню, которое имеет следующую структуру:
И следующий CSS:
.menu_block{...}
.menu{...}
.menu_item{...}
.item{...}
.item:hover{...}Первым, что хочется спросить, смотря на этот код, это зачем происходит вложенность тега ul в div? Тег ul, как и тег div относятся к блочным элементам, поэтому в данном случае тег ul не требуется оборачивать тегом div.
Далее, у меня была очень вредная привычка добавлять к классам «пояснения» вроде _block, _list и т.п. Этот метод не помогает вашему коду быть прозрачнее. Это лишняя тавтология, которая бессмысленна и может легко сломаться. Поэтому, изменю код следующим образом:
.menu{...}
.menu_item{...}
.item{...}
.item:hover{...}Далее, посмотрим на ссылку с классом item. Данное название класса очень обобщенное. Если бы через полгода меня спросили для чего я использую этот класс, то я бы не ответил. Я бы не ответил даже тогда, когда верстал этот макет, если бы не посмотрел в HTML. Запомните, что название классов должно отражать смысл элемента.
Теперь изменим код:
.menu{...}
.menu__item{...}
.menu__link{...}
.menu__link:hover{...}Вот теперь, если даже в 4 ночи позвонят мне и спросят зачем там класс menu__link, то я отвечу для стилизации ссылок в меню.
Поиск
Теперь рассмотрим поиск со следующей структурой:
Как видим, здесь, как и ранее, есть проявлении моей болезни тавтологии и лишней вложенности блочного элемента form в блочный div. Я надеюсь, что теперь вы знаете как решать такие проблемы, поэтому я оставлю это вам. Должны же вы потренироваться! Но если вы не хотите, ответ в следующем абзаце.
Кроме перечисленных выше ошибок, в этом фрагменте есть еще несколько. Первая, это не указан атрибут name для поля ввода.
Вторая, это название класса search_icon для кнопки поиска. Опять же, если меня спросят для чего служит этот элемент, то я снова не назову правильный ответ. Поэтому изменим класс на search__button.
Третья, использование декоративной иконки с лупой в HTML. Почему это ошибка? Потому что, все вставленные изображения на страницу при помощи тега img будут проиндексированы поисковыми роботами. Особого смысла индексировать элементы декора я не вижу, поэтому, советую в HTML оставлять только изображения, которые относятся к контенту страницы.
Теперь изменим HTML.
Вроде теперь все хорошо… Но, стоп! У нас теперь абсолютно пустая кнопка! Это очень плохой тон в верстке. Если у нас не загрузится CSS или пользователь будет посещать сайт с использование электронных читалок, то они просто ее не увидят или не услышат. Поэтому нам нужно добавить слово «Найти», которое мы скроем используя свойство text-indent.
Теперь перейдем к CSS.
.top-panel .search{...}
.top-panel .search .search_field{...}
.top-panel .search .search_field:focus{...}
.top-panel .search .search_button{...}Как вы теперь видите, я указал всю цепочку вложенности в CSS-селекторе. Мне постоянно встречаются такие ошибки. Давайте рассмотрим подробнее эту ошибку.
Первое, мы привыкли с вами читать слева направо, и поэтому думаем, что браузер сначала найдет элемент с классом top-panel, а потом внутри него элемент с классом search. Но, на деле все наоборот! Браузеры читают CSS-селектор справа налево, поэтому, сначала он найдет элементы с классом search, а только потом у них отыщет предков с классом top-panel.
Второе, все начинающие верстальщики думают, что сверстают сайт и его больше никто не тронет. И будет он жить в таком состоянии всю свою жизнь. Но, на самом деле верстка в первозданном виде живет ровно до того момента, пока до нее не дотронуться программисты.
Но, это полбеды. Если вы верстаете мало мальски посещаемый сайт, то рано или поздно придется вносить изменения на нем. Допустим, требуется перенести блок поиска в другое место страницы. Например, в подвал. При текущей реализации потребуется изменить как HTML, так и CSS. Это все потому, что указана жесткая вложенность элементов. Вы вынуждены будете продублировать CSS-свойства.
Для решения этой проблемы нам нужно изменить CSS следующим образом:
.search{...}
.search__field{...}
.search__field:focus{...}
.search__button{...}Все просто! Теперь мы можем перемещать поиск, поле поиска и даже кнопку куда захотим.
Позиционирование меню и поиска
Теперь перейдем к рассмотрению позиционирования меню и поиска. По дизайну отчетливо видно, что меню прижато к левой части основного контейнера, а поиск — к правой. Очевидно, что требуется использовать float-элементы. Текущая реализация выглядит так:
.top-panel{
background-color: #ededed;
min-width: 960px;
height: 40px;
}
.menu{
float: left;
}
.search{
float: right;
}В чем же здесь ошибка? Из-за того что у элемента с классом top-panel задана высота в 40 пикселей, блоки выглядят как надо. Но, если мы проинспектируем элемент с классом main-container при помощи веб-инспектора, то увидим, что его высота равняется нулю. Это побочный эффект при использовании float-элементов. Что можно сделать с этим? По логике начинающего верстальщика, надо просто задать высоту. И таким образом появится что-то вроде этого:
.top-panel{
background-color: #ededed;
min-width: 960px;
}
.top-panel .main-container{
height: 40px;
}Основная проблема заключается даже не во вложенности каскада, а в том, что указано фиксированное значение для высоты элемента с классом main-container. В верстке высота блока должна подстраивается под контент элемента. Поэтому ее нельзя фиксировать.
Хорошо, нельзя задавать высоту, но как же исправить эту ситуацию? Есть очень простой и надежный способ. Это добавить элемент со свойством clear и значением both за float-элементами или в конец родительского блока, как в нашем случае.
Для добавления такого элемента воспользуемся псевдоэлементом : after.
.clearfix:after{
content: "";
display: block;
clear: both;
}И HTML примет следующий вид:
CSS изменится следующим образом:
.top-panel{
background-color: #ededed;
min-width: 960px;
}
.menu{
float: left;
}
.search{
float: right;
}А теперь, подумаем над следующим вопросом. Мы уже говорили, что при правильной верстке блоки можно свободно перемещать по странице. Но, при текущей реализации мы не добьемся такой универсальности из-за свойства float. Как же быть? Мы точно знаем, что в верхней панели меню находится слева, а поиск — справа. Поэтому можно создать дополнительные классы, в которых и пропишем float:
CSS изменится следующим образом:
.top-panel{
background-color: #ededed;
min-width: 960px;
}
.top-panel__menu{
float: left;
}
.top-panel__search{
float: right;
}Стилизация меню
Для стилизации блока я использую следующий CSS:
.top-panel__menu{
float: left;
}
.menu{
width: 620px;
padding: 12px 0;
list-style: none;
}
.menu__item{
display: inline-block;
}
.menu__link{
color: #353743;
font-size: 14px;
margin-right: 15px;
text-decoration: none;
}
.menu__link:hover{
text-decoration: underline;
}Какие ошибки я здесь вижу.
Во-первых, у menu задан padding сверху и снизу равные в 12 пикселей. Это нужно для того, чтобы сделать отступы от края блока. Вроде все хорошо. Но, если мы посмотрим на верхнюю панель в целом, то увидим, что у поиска тоже есть отступ сверху. Поэтому, общий отступ лучше задавать родительскому блоку. Например, .top-panel. А у меню добавить не хватающие значения, используя уже margin, т.к это уже будет внешний отступ:
.top-panel{
background-color: #ededed;
min-width: 960px;
padding-top: 7px;
padding-bottom: 7px;
}
.top-panel__menu{
float: left;
}
.menu{
width: 620px;
margin-top: 5px;
margin-bottom: 5px;
list-style: none;
}Во-вторых, в данном случае у меню не нужно указывать ширину. Вы же помните, что верстка в своем начальном состоянии живет недолго? На живом сайте может быть больше пунктов меню, поэтому оно должно свободно увеличиваться.
В-третьих, отступы между пунктами меню почему-то заданы у ссылок, хотя логичнее сделать это у элементов li. Вроде мелочь? Но в верстке мелочей не бывает!
Измененный CSS меню теперь выглядит таким образом:
.top-panel{
background-color: #ededed;
min-width: 960px;
padding-top: 7px;
padding-bottom: 7px;
}
.top-panel__menu{
float: left;
}
.menu{
margin-top: 5px;
margin-bottom: 5px;
list-style: none;
}
.menu__item{
display: inline-block;
margin-right: 15px;
}
.menu__link{
color: #353743;
font-size: 14px;
text-decoration: none;
}
.menu__link:hover{
text-decoration: underline;
}Стилизация поиска
Для стилизации блока я использую следующий CSS:
.top-panel__search{
float: right;
}
.search{
padding: 5px 0;
}
.search__field{
width: 135px;
height: 25px;
background-color: #D2D2D2;
padding: 0 10px;
font-size: 12px;
transition: width 0.9s 0s;
vertical-align: middle;
}
.search__field:focus{
width: 200px;
outline: none;
}
.search_button{
position: relative;
right: 4px;
top: 1px;
background: none;
vertical-align: middle;
}Из-за того, что я добавил padding к элементу с классом top-panel, у нас увеличилось расстояние от верхнего края панели до поиска. Поэтому, я удалю его из класса search.
Также, ранее, мы изменили верстку кнопки поиска, поэтому, на данном этапе она отличается от дизайна. Нам нужно задать размеры кнопки и добавить иконку лупы, используя свойство backgorund. Также требуется установить text-indent для скрытия текста:
.search_button{
width: 25px;
height: 25px;
position: relative;
right: 4px;
top: 1px;
text-indent: -9999px;
background: url("../images/search_icon.png") no-repeat 0 0;
vertical-align: middle;
}Теперь разберем ошибки. Первым, что бросается в глаза — это использование position в кнопке. Если мы отключим position: relative в веб-инспекторе, то увидим расстояние между полем ввода и кнопкой. Оно появилось, потому что и поле ввода, и кнопка являются строчно-блочными элементами.
Для решения это проблемы, я просто задам им свойство float со значением left и уберу position у кнопки:
.search__field{
width: 135px;
height: 25px;
background-color: #D2D2D2;
padding-right: 10px;
padding-left: 10px;
font-size: 12px;
transition: width 0.9s 0s;
float: left;
vertical-align: middle;
}
.search_button{
width: 25px;
height: 25px;
float: left;
text-indent: -9999px;
background: url("../images/search_icon.png") no-repeat 0 0;
vertical-align: middle;
}Далее, добавлю ранее созданный класс clearfix к элементу с классом search:
И вроде как работает. Но! У нас сейчас есть одна из моих любимейших ошибок, которую я встречаю даже у опытных верстальщиков. Давайте посмотри вновь на CSS поля ввода и кнопки:
.search__field{
width: 135px;
height: 25px;
background-color: #D2D2D2;
padding-right: 10px;
padding-left: 10px;
font-size: 12px;
transition: width 0.9s 0s;
float: left;
vertical-align: middle;
}
.search_button{
width: 25px;
height: 25px;
float: left;
text-indent: -9999px;
background: url("../images/search_icon.png") no-repeat 0 0;
vertical-align: middle;
}В обоих классах есть свойство vertical-align, с помощью которого можно выровнять строчно-блочные элементы по вертикальной оси. Но! Сейчас у нас уже блочные элементы! Свойство float влияет на поток элементов и делает их блочными. Это нужно помнить всегда. Поэтому я удалю vertical-align у элементов.
.search__field{
width: 135px;
height: 25px;
background-color: #D2D2D2;
padding-right: 10px;
padding-left: 10px;
font-size: 12px;
transition: width 0.9s 0s;
float: left;
}
.search_button{
width: 25px;
height: 25px;
float: left;
text-indent: -9999px;
background: url("../images/search_icon.png") no-repeat 0 0;
}Заключение
Когда я начал писать статью, я не думал, что описание своих ошибок будет занимать такой объем текста. Поэтому, если вы хотите продолжить разбор ошибок следующий блоков, то напишите мне об этом в комментариях.
Комментарии (14)
lostpassword
8 августа 2016 в 17:01
0↑
↓
Подскажите, почему скрывать текст лучше через text-indent, а не через font-size:0?
melnik909
8 августа 2016 в 17:03
0↑
↓
Слушал подкаст «Веб-стандарты», и там Вадим Макеев говорил про text-indent. Я хочу проверить это, но пока руки не дошли. Поэтому доверяем старшему поколению =)P.S: Если у вас есть возможность проверить, то давайте протестируем.
-
lostpassword
8 августа 2016 в 17:27 (комментарий был изменён)
0↑
↓
Нет, спорить с вами не было намерения — просто хотел узнать, какие есть плюсы.)-
melnik909
8 августа 2016 в 17:28 (комментарий был изменён)
0↑
↓
Нашел вот такой тест — http://terrillthompson.com/tests/hiddencontent.html
allard
8 августа 2016 в 17:38
0↑
↓
Еще есть момент по поводу text-indent: -9999 px;, давно уже отказался от -9999 px. В свое время где-то встречал, что это лишняя нагрузка на браузер. Вместо этого использую:text-indent: 100%; white-space: nowrap; overflow: hidden;
8 августа 2016 в 17:46
0↑
↓
насколько помню, были проблемы в Safari браузере с font-size:0, но могу ошибаться.
miraage
8 августа 2016 в 17:09
+1↑
↓
Почемуul > li > aвместоnav > a?-
melnik909
8 августа 2016 в 17:15 (комментарий был изменён)
0↑
↓
Потому что первоначально меню было сверстано через список, но если не учитывать это, то использование тега nav более правильное решение. 
Synoptic
8 августа 2016 в 17:20
+1↑
↓
Не в упрек статье, сам такое писал. Я понимаю, что первое время может быть сложно, но лучше сразу выберите одну из существующих методологий верстки, например БЭМ (но не обязательно ее, главное — чтобы логика верстки подчинялась каким-то формализованным правилам) и используйте ее.Не пугайтесь документации того же БЭМ, для верстальщика там знать нужно минимум — ключевые понятия и как решаются стандартные проблемы при верстки этим способом.
Это будет лучше любого самодельного велосипеда, реализующего тоже самое, только хуже.
-
melnik909
8 августа 2016 в 17:25
0↑
↓
Извините, но не понял про велосипед. Я в работе сейчас и использую БЭМ. В статье специально не упоминал про него, потому что цель была показать ошибки.-
Synoptic
8 августа 2016 в 17:33
0↑
↓
Большая часть перечисленных ошибок проистекает из-за отсутствия заранее определенных правил верстки. Методология задает такие правила.-
melnik909
8 августа 2016 в 17:35
0↑
↓
И да, и нет. Начинающему специалисту практически невозможно выбрать методологию. Но я основываюсь на своем опыте.-
Synoptic
8 августа 2016 в 17:38 (комментарий был изменён)
0↑
↓
Можно выбрать любую из популярных не вдаваясь в подробности — БЭМ, OOCSS, изучить ее, и верстать в этом стиле. Для начинающего, да и для опытного верстальщика это будет лучше, чем если не выбирать никакую.-
melnik909
8 августа 2016 в 17:46
0↑
↓
Я согласен, что методологии нужны. Но начинающему надо изучать семантику, способы позиционирования и другие основы.
