search

enb / enb-bh

BH support for ENB
Other
15 stars 12 forks source link

Suport BH@v4.x #30

Closed mishanga closed 9 years ago

mishanga commented 9 years ago

https://github.com/bem/bh/releases/tag/v4.0.0

dfilatov commented 9 years ago

Is there any progress? It's really difficult to use the newest bh without this support.

bratva commented 9 years ago

+1 we need it

tadatuta commented 9 years ago

blocked by issue with tests: https://github.com/enb-bem/enb-bh/pull/31

tsofist commented 9 years ago

@blond Андрей, я так понимаю изменение пойдет в ближайший tag? А не подскажете когда его планируете выкатить?

blond commented 9 years ago

@tsofist, изменение войдёт в v1.0.0.

Оставшиеся задачи — https://github.com/enb-bem/enb-bh/milestones/1.0

Задач не много, но это займёт какое-то время. Основное это документация и руководство по миграции.

Если очень нужно прямо сейчас, могу выпустить enb-bh@1.0.0-alpha1.

tsofist commented 9 years ago

@blond, спасибо. Вроде не горит, так что не к чему вас торопить - подождём.

mishanga commented 9 years ago

Мне горит :) Буду несказанно благодарен за альфу.

tadatuta commented 9 years ago

Может сначала сделать https://github.com/enb-bem/enb-bh/issues/26, чтобы альфу можно было и на коде islands потестить?

blond commented 9 years ago

@mishanga, выпустил enb-bh@1.0.0-alpha1.

@tadatuta выпустить вторую альфу после добавления #26 не проблема ;)

mishanga commented 9 years ago

@blond а ты её в npm не опубликовал еще?

blond commented 9 years ago

@mishanga опубликовал, конечно.

При установке юзай тег alpha:

$ npm i enb-bh@alpha

или точную версию:

$ npm i enb-bh@1.0.0-alpha1
mishanga commented 9 years ago

простите, туплю

arikon commented 9 years ago

@tadatuta Разве jsElem нужно в islands?

blond commented 9 years ago

@arikon по умолчанию bh@4.0.0 добавляет i-bem класс к элементам.

В islands нужно, чтобы i-bem добавлялся только к блокам. https://github.com/bem/bh/pull/148

arikon commented 9 years ago

@blond Понял, спасибо