Оцените очередную верстку

[IvanRu08]

Всем привет. Давно выкладывал в этот раздел такую работу. Набрался немного опыта, решил переверстать. Хочу выслушать вашу критику по поводу качества кода, багов и так далее. Заранее спасибо.http://a96362zl.bget.ru/index.html

[DivMan]

А зачем ради того, что бы скрыть поле поиска, подключать jquery?

[kent]

Вместо спрайтов http://fontello.com/

[VeniVidiVici]

Из советов к вашей предыдущей теме:

Цитата

В таком случае пишем, то margin: 0 24px; для каждого блока, но у общего контейнера отнимает это же margin: 0 -24px;. В таком случае не нужны :nth-child(3). А идеально было бы задать контейнеру класс, к примеру, .three-column-wrapper, а содержимому просто .column.

К .news снова применяете :nth-child(3). Почему бы и не воспользоваться хорошим советом из предыдущей темы?

Цитата

Проще сделать спрайт с белыми значками соц. контактов, а цвет фона задавать через стили. Универсальность - 100%

В футере, снова делаете иконки 2 состояний, хотя хватило бы и белых, снова совет из предыдущей темы. 

Edited May 16, 2016 by VeniVidiVici

[IvanRu08]

9 часов назад, DivMan сказал:

А зачем ради того, что бы скрыть поле поиска, подключать jquery?

В дальнейшем на сайте с помощью jquery будет реализован не только поиск.

 

6 часов назад, VeniVidiVici сказал:

Из советов к вашей предыдущей теме:

К .news снова применяете :nth-child(3). Почему бы и не воспользоваться хорошим советом из предыдущей темы?

У меня этот блок выравнивает с помощью margin 0 auto. Придется создавать еще один div внутри него и прописывать эти стили или как? Вот, попробовал. Создал еще один контейнер внутри и прописал стили. Справа появился вообще какой то непонятный отступ. Что не так делаю?http://a96362zl.bget.ru/index.html

[VeniVidiVici]

Да, с margin странные вещи, ну вот с паддингом, как в бутстрапе:

код

Вообще, если у тебя один ряд, то проще всего будет написать так:

.news + .news {
	margin-left: 17px;
}

и ничего обнулять не надо.

 

Edited May 17, 2016 by VeniVidiVici