@trydim

Оцените верстку по БЭМ?

Здравствуйте.. Уделите пожалуйста 5-7 мин своего драгоценного...

Новый верстальщик выполнил первое задание..
Утверждает что так правильно по БЭМ методологии.. возник некоторый спор. Хотелось бы услышать пару слов от опытных верстальщиков, т.к. сам на бэкэнд и версткой почти не занимался (только азы).

https://github.com/Trydim/testhtml

Большое спасибо за внимание.
  • Вопрос задан
  • 191 просмотр
Пригласить эксперта
Ответы на вопрос 3
sniggering_deus
@sniggering_deus Куратор тега Вёрстка
Dance Gavin Dance - Prisoner
Определённая доля БЭМ конечно имеется, но как по мне, есть моменты которые вообще не по БЭМ.

Например имена блоков/элементов должны быть осмысленными, но вот это сложно назвать чем-то нормальным:

<span class="calc__span">200 мм</span>

И такие блоки повторяются кучу раз:

<span class="advantages-item__span">
Срок службы штакетника из оцинкованной рулонной стали до 50 лет
</span>


Что ещё бросается в глаза:

<div class="slider-mini">
<button class="slider-mini__btn_prev" type="button"></button>
<img src="" class="" alt="">
<img src="" class="" alt="">
<img src="" class="" alt="">
<img src="" class="" alt="">
<button class="slider-mini__btn_next" type="button"></button>
</div>


Отсутствие классов у изображений в слайдере. Возможно они просто ещё не проставлены.

Кода слишком много и расписывать про каждый косяк долго, поэтому в принципе можете и сами ознакомиться с данной методологией на официальном сайте и сравнивать всю верстку с примерами из документации. Сайт. Я думаю это будет намного лучше, к тому же много времени это не займет.
Ответ написан
wapster92
@wapster92 Куратор тега Вёрстка
<div class="heading-block">
                <h3 class="heading__h3">Продажа, производство и доставка</h3>
                <h1 class="heading__h1">Штакетник металлический</h1>
              </div>
не БЭМ
<div class="grid-block">
                            <span class="calc__span">Стоимость погонного метра</span>
                            <span class="calc__text">61 ₽/м </span>
                            <span class="calc__span">
тоже самое, и куча подобных моментов, где элемент вложен в чужой блок, если так делать то только при миксе. Про css ничего не могу сказать, там похоже постпроцессоры применены.
Upd все же могу и про css сказать. Блокам задан внешний отступ, в бэме этого делать так же нельзя
.header-item {
  margin-right: 47px
}
Ответ написан
kirbi1996
@kirbi1996
В целом бэм используется, однако есть странные моменты когда допустим к span применяет еще класс
calc_span, но если у него небольшой опыт то думаю что не плохо. По поводу css судя по тому что нет точки с запятой то там точно компилировалось, думаю что это не плохо. Качество верстки с телефона не смог посмотреть к сожалению, но если верстальщик начинающий то думаю можно направить в нужное русло
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы