RusRogo / RusRogo.github.io

Portfolio
0 stars 0 forks source link

BEM Naming #1

Open ZvezdocheTag opened 6 years ago

ZvezdocheTag commented 6 years ago

Снова по бем, смотри должен быть обособленный блок типа (list, header, list), и в этом блоке должны быть элементы которые несут смысловую нагрузку и привязаны к блоку родителю. К примеру, у нас есть блок : header В нем у нас по макету есть, логотипы, какой то слоган компании к примеру, соответственно мы создаем два блока с классами : header__logo, header__slogan.

Пару моментов, для обособленный секций в html есть тег section https://github.com/RusRogo/RusRogo.github.io/blob/78d4d6df49f5015440cb497e3a25bb00568cae7e/index.html#L10 Возмес у тебя блок class="wrapper wrapper__header", смотри wrapper это блок, в твоем коде он замешан с элементом в одном классе, такого не должно быть, если хочешь замодифицировать блок добавляешь к нему модификатор wrapper wrapper_top , wrapper wrapper_bottom. Но опять же у тебя каждый wrapper, это обособленный блок, который имееть общую разметку, то есть ты можешь добавить два блока:

<section class="wrapper community">...</section>
<section class="wrapper info">...</section>
<section class="wrapper etc">...</section>
RusRogo commented 6 years ago

В нем у нас по макету есть, логотипы, какой то слоган компании к примеру, соответственно мы создаем два блока с классами : headerlogo, headerslogan.

Ты имел ввиду - 2 элемента?

RusRogo commented 6 years ago

Переделал так - <section class="wrapper wrapper_header">

ZvezdocheTag commented 5 years ago

так приятней