jsAttr Defaults: `data-bem` name + `json` scheme

[blond]

• [ ] Use data-bem value instead of onclick for jsAttrName option.
• [ ] Use json value instead of js for jsAttrScheme option.

[mishanga]

Зачем?

[sladex]

data-bem + json более реалистичная и, наверное, распространенная связка. Стоит ли ради этого менять значения по умолчанию? Кому-то после такого обновления придется править восстанавливать старые значения.

Но если смотреть на bh отдельно то data-bem и json навязанные технологии и, в отличии от onclick и js, "не работают из коробки". Мои личные ощущения :).

[blond]

Зачем?

Эти значения нужны для корректной работы с bem-core. Кажется, что уже пора рекомендовать по умолчанию bem-core. К тому же другие опции работают по умолчанию так же как в bem-core, например, jsElem: true.

Стоит ли ради этого менять значения по умолчанию? Кому-то после такого обновления придется править восстанавливать старые значения.

Да, это мажорное изменение.

Но если смотреть на bh отдельно то data-bem и json навязанные технологии и, в отличии от onclick и js, "не работают из коробки".

Что ты имеешь ввиду под не «работают из коробки»? И почему onclick и js работают?

[mishanga]

Ради одной библиотеки я не буду менять дефолты. bem-core — не эталон, у BH много других потребителей, которые используют другие настройки.

[tadatuta]

@mishanga почему одной? в bem-bl аналогично == у всех польхователей Islands. тут скорее вопрос в том, остался ли вообще хоть кто-нибудь со старым вариантом и при этом на BH?

[tadatuta]

@mishanga Миш, так есть ли пользователи, которым бы нужен был старый дефолт?

[sipayRT]

подпишусь на тред. Мне тоже интересно сколько людей используют старый вариант