Proposal: ctx.if()

[Yeti-or]

Добавить сахара: вместо:

ctx
  .mix({ elem : 'control' })

if (ctx.mod('disabled') {
 ctx.attr('tabindex', -1)
} else {
 ctx.attr('title', 'yeah')
}
ctx.js(false)

писать что то типа такого:

ctx
  .mix({ elem : 'control' })
  .if(ctx.mod('disabled'), function(predicate) {
      if (predicate) {
        ctx.attr('tabindex', -1)
      else 
        ctx.attr('title', 'yeah')
  })
  .js(true)

[Yeti-or]

ради чейнинга всё

[megatolya]

кажется только более нечитаемо станет)

[mishanga]

Мне тоже не нравится. На самом деле тебе ничто не мешает расширить Ctx у себя на проекте.

[qfox]

@mishanga Как на счет добавить сахара для расширения Ctx? Сейчас оно расширяется с помощью utils, который создается динамически, и все лежит в одном файле. Не думал про разнесение на три файла: сам BH, Ctx, и все, что связано с toHTML?

[mishanga]

Не вижу смысла усложнять всё. Задача решается тривиально:

bh = new BH();
bh.utils.if = function(expr, fnTrue, fnFalse) {
    if (expr) fnTrue && fnTrue();
    else fnFalse && fnFalse();
    return this;
}