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

i-bem: trigger twice runs jquery-trigger

Open baymer opened this issue 11 years ago • 3 comments

И __base, и __self.trigger дважды вызывают вот тут: https://github.com/bem/bem-bl/blame/dev/blocks-common/i-bem/i-bem.js#L193

вот это: https://github.com/bem/bem-bl/blob/dev/blocks-common/i-jquery/__observable/i-jquery__observable.js#L167

Не совершается ли лишняя работа?

baymer avatar Feb 03 '15 12:02 baymer

@dfilatov боюсь, здесь не обойтись без твоего мнения. Посмотришь?

deeonis avatar Feb 25 '15 16:02 deeonis

__base вызывает триггер из https://github.com/bem/bem-bl/blob/dev/blocks-common/i-jquery/__observable/i-jquery__observable.js#L167, __self.trigger(e, data); вызывает триггер на классе. В чем именно тут лишняя работа?

dfilatov avatar Feb 25 '15 16:02 dfilatov

@baymer :eyes:

deeonis avatar Feb 25 '15 16:02 deeonis