bem-xjst icon indicating copy to clipboard operation
bem-xjst copied to clipboard

Invert `xjstOptions` `flush`

Open indutny opened this issue 8 years ago • 4 comments

def() should return result of other apply, without any changes or prefixing with other string.

indutny avatar Sep 14 '15 09:09 indutny

cc @veged

indutny avatar Sep 14 '15 09:09 indutny

@indutny What is this task about? Why is it needed?

arikon avatar Dec 30 '15 15:12 arikon

@arikon right now blocks with custom renderer (in other words def() predicate), will turn off flushing behavior for all child blocks. xjstOptions is used to re-enable flushing explicitly on some templates with def(), assuming that people know what they are doing.

@veged suggested to flush all templates by default, and allow people to disable this behavior using xjstOptions. Thereby inverting current API.

indutny avatar Dec 31 '15 05:12 indutny

@tadatuta @zxqfox @veged can anybody explain to me why we still need flush option?

P.S. Issue resolved.

miripiruni avatar Sep 20 '16 14:09 miripiruni