search

enb / enb-bem-i18n

BEM internationalization for ENB
Other
8 stars 7 forks source link

enb-bem-i18n: new description #45

Closed innabelaya closed 9 years ago

innabelaya commented 9 years ago

Resolved #20

innabelaya commented 9 years ago

@blond ping?

blond commented 9 years ago

Написал замечаний. Про шаблоны и API технологий, давай писать после того как доделаем это в коде.

В целом мне очень нравится :+1:

innabelaya commented 9 years ago

@blond внесла правки с последней встречи

blond commented 9 years ago

По всему документу библиотека называется BEM.i18n. Предлагаю называть везде просто i18n, а BEM.I18N писать только когда речь о bem-bl.

blond commented 9 years ago

Есть мысли на подумать: очень многое описывает только bem-bl специфику, очень многое bem-core. Может это должно как-то сказаться на структуре документа? Например, разделить документ на 2 независимых, или вначале описать всё общее, а потом всё частное, или же каждый подраздел явно делить на два.

Но я не уверен :)

blond commented 9 years ago

Написал замечания. Пока что не совсем уверен в структуре документа, например не хватает раздела про API и не понятно что делать с вечной историей про разное с bem-bl и bem-core.

Описание технологий пока не читал, думаю логично добить эту часть, когда с остальным будет понятно.

tormozz48 commented 9 years ago

@blond А этот pr должен быть влит в мастер или в 1.x ?

blond commented 9 years ago

А этот pr должен быть влит в мастер или в 1.x ?

В 1.x

tormozz48 commented 9 years ago

В 1.x

А сейчас смотрит в мастер. Нужно будет перенаправить

blond commented 9 years ago

Написал замечания.

blond commented 9 years ago

@innabelaya всё супер! :+1:

Можешь после правок переоткрыть PR в ветку 1.x?

blond commented 9 years ago

/cc @tadatuta

blond commented 9 years ago

Можешь после правок переоткрыть PR в ветку 1.x?

@innabelaya 1.x превратился в master. Нужно только отребейзить.

innabelaya commented 9 years ago

@blond правки внесла

blond commented 9 years ago

Closed in favor of #68