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

jsAttr Defaults: data-bem name + json scheme #147

Open
2 tasks
blond opened this issue May 24, 2015 · 7 comments
Open
2 tasks

jsAttr Defaults: data-bem name + json scheme #147

blond opened this issue May 24, 2015 · 7 comments

Comments

@blond
Copy link
Member

blond commented May 24, 2015

  • Use data-bem value instead of onclick for jsAttrName option.
  • Use json value instead of js for jsAttrScheme option.
@mishanga
Copy link
Member

Зачем?

@sladex
Copy link

sladex commented Jun 17, 2015

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

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

@blond
Copy link
Member Author

blond commented Jun 17, 2015

Зачем?

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

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

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

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

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

@mishanga
Copy link
Member

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

@tadatuta
Copy link
Member

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

@tadatuta
Copy link
Member

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

@tadatuta tadatuta reopened this Aug 18, 2015
@sipayRT
Copy link
Contributor

sipayRT commented Aug 19, 2015

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

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

5 participants