search

sbmaxx / bem-priv

Use the power of inherit.js to generate BEMJSON with BEMPRIV blocks
21 stars 6 forks source link

Добавление параметров js, mods, content и т.д. #14

Closed DimitryDushkin closed 9 years ago

DimitryDushkin commented 10 years ago

Например, если надо добавить какое-то свойтво в mods, то надо писать так:

this.mods(_.assign(this.mods, { newProp: value });

Хорошо бы какой-нибудь хелпер для таких ситуаций, ну или прокачать mods(), чтобы он понимал, что надо расширить объект, а не заменять его.

cherninely commented 10 years ago

Дима :))) https://github.com/sbmaxx/bem-priv/issues/11

DimitryDushkin commented 10 years ago

Илья)) Если бы можно было бы по одному выставлять, уже было бы хорошо.

cherninely commented 10 years ago

я то вообще за :) Думаю Рома ответит что то типа "я принимаю пулреквесты" :D

DimitryDushkin commented 10 years ago

Надо еще учесть, что js может быть true, в этом случае надо создавать объект.

DimitryDushkin commented 10 years ago

Ок, тогда ждём "за" Ромы и я пулл реквестну.

sbmaxx commented 10 years ago

@DimitryDushkin а что не так с this.mod? Как раз для проставления одного модификатора. Я не очень понял идею ;)

DimitryDushkin commented 10 years ago

@sbmaxx mod правильно работает, да. Хочется, чтобы так работали js() и content().

sbmaxx commented 10 years ago

а как ты себе это представляешь с content'ом? который может быть любым типом данных?

DimitryDushkin commented 10 years ago

Да, с content(), похоже, никак. Ну тогда только js()?