i-bem__dom.js меняет стандартное поведение

bem / bem-mvc

Yet another MVC for i-bem

29 stars 26 forks source link

i-bem__dom.js меняет стандартное поведение #171

Closed kompolom closed 8 years ago

kompolom commented 8 years ago

При использовании пакета с bem-core (v2.8) и bem-mvc (v2). Появляется следующая проблема:

var insertedDom = BEMDOM.append(node, htmlStr); //undefined

Методы изменяющие dom перестаю возвращать изменения. А это очень полезная фича. Нужно ли переопределение блока i-bem__dom на уровне bem-mvc?

dosyara commented 8 years ago

Ветка v2 очень слабо поддерживается и bem-core очень далеко ушел вперед по сравнению с текущей версией в v2. Какое-то API может быть сломано. Никаких специальных ограничений для bem-core со стороны bem-mvc я не помню, скорее всего это просто баг из-за разницы в версиях. Если есть возможность, присылайте PR с фиксом.

tadatuta commented 8 years ago

Насколько я вижу, должно быть достаточно просто добавить return перед https://github.com/bem/bem-mvc/blob/v2/common.blocks/i-bem/__dom/i-bem__dom.js#L12