redKOTik / fsd-frontend-second-task

MIT License
0 stars 0 forks source link

BEM #12

Open Feinminen opened 3 years ago

Feinminen commented 3 years ago

Название папки/файла/миксина/класса и блока всегда должны совпадать. В папке конкретного блока должен лежать ТОЛЬКО один pug, scss и если нужно js файл. Делить элементы от блока на разные файлы лучше не нужно, сложнее вложенность но при этом переиспользоваться они могут только внутри этого блока, поэтому их имеет смысла разделять. Также один файл - это всегда только один миксин.

В БЭМ не может быть элемента без блока или модификатора без элемента. Сейчас много мест, где всё начинается с элемента, а блока не существует.

image Нельзя чтобы в одном файле было несколько блоков, блок всегда должен быть один.

В целом с именованием всё норм, но встречается и неправильное именование - название блока/элемента должнно описывать сущность и отвечать на вопрос что это, а модификаторы должны описывать состояние (какая это сущность). тык

Также порой используются крайне не семантичные названия, e.g. 'span',

Сейчас есть блоки, которые явно должны были быть просто модификациями одного блока, например есть bullet-list и bullet-list-colored, даже исходя из названия видно, что второе должно быть модификатором первого. Также обрати внимание на порядок слов 'bullet-list-colored' грамматически неправильно, должно быть 'colored-bullet-list' если на это смотерть именно как на словосочетание.