Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

БЭМ #6

Open
Everrest1 opened this issue Jun 9, 2020 · 10 comments
Open

БЭМ #6

Everrest1 opened this issue Jun 9, 2020 · 10 comments

Comments

@Everrest1
Copy link

Что такое _form? Если это имя блока, то почему использовано нижнее подчёркивание, а не дефис?
image

@Everrest1
Copy link
Author

По стандартам п.1 https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/bem.md

блок, элемент — всегда имя существительное (noun);

image

@Everrest1
Copy link
Author

Элемент модификатора элемента?

image

@Everrest1
Copy link
Author

Не стоит использовать такие длинные имена нигде

Модификатор size, судя по имени, должен быть булевым, тогда значение должно отделяться от имени двумя дефисами.

block-name__elem-name--mod-name--mod-val

image

@Everrest1
Copy link
Author

А вообще, рекомендую тебе использовать стандартную схему именования, с ней должно быть меньше путаницы

ivanushkaPr added a commit that referenced this issue Jun 10, 2020
ivanushkaPr added a commit that referenced this issue Jun 10, 2020
@ivanushkaPr
Copy link
Owner

А вообще, рекомендую тебе использовать стандартную схему именования, с ней должно быть меньше путаницы

Изменил имена блоков в соответствие со стандартной схемой именования.

@ivanushkaPr
Copy link
Owner

По стандартам п.1 https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/bem.md

блок, элемент — всегда имя существительное (noun);

image

Исправил.

@ivanushkaPr
Copy link
Owner

ivanushkaPr commented Jun 16, 2020

Модификатор size, судя по имени, должен быть булевым, тогда значение должно отделяться от имени двумя дефисами.

block-name__elem-name--mod-name--mod-val

image

Во время изменения правил именования по БЭМ, я сверился с информацией на сайте: https://ru.bem.info/methodology/naming-convention/

Не нашел там информации о необходимости разделения модификаторов ключ - значения(как я понимаю вы их имели ввиду, а не булевые) двумя дефисами при классической схеме наименования, там пишут про один один дефис.

Стиль doubleDash альтернативный, а вы писали про стандартную схему.
В любом случае, все модификаторы типа ключ - значение были разделены дефисом.

P.s. Какой стиль именования используется в FSD - стандартный или альтернативный double dash?

@Everrest1
Copy link
Author

Не нашел там информации о необходимости разделения модификаторов ключ - значения(как я понимаю вы их имели ввиду, а не булевые) двумя дефисами при классической схеме наименования, там пишут про один один дефис.

Да, ключ-значение, конечно же. Так как у тебя в проекте использовался альтернативный стил, я для него и указал)
Вот это имел ввиду
image

Классический в FSD
image

@Everrest1
Copy link
Author

Everrest1 commented Jun 17, 2020

По стандартам п.1 https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/bem.md

модификатор — должен удовлетворять свойствам модификатора из английского языка. Всегда adjective или adjectival phrase

image
image

И так далее во всём проекте

@ivanushkaPr
Copy link
Owner

Не нашел там информации о необходимости разделения модификаторов ключ - значения(как я понимаю вы их имели ввиду, а не булевые) двумя дефисами при классической схеме наименования, там пишут про один один дефис.

Да, ключ-значение, конечно же. Так как у тебя в проекте использовался альтернативный стил, я для него и указал)
Вот это имел ввиду
image

Классический в FSD
image

Понял, спасибо. Выходит я сам и себя, и тебя запутал =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants