Re-evaluation of translate pipes if ChangeDetectionStrategy.OnPush is set

[ngx-translate]

Re-evaluation of translate pipes if ChangeDetectionStrategy.OnPush is set

[ocombe]

I'm not expert enough in Angular 2 to know the right thing to do here. Do you have an article somewhere explaining the different change detection strategies ?
And an example app that uses OnPush ?

[hpop]

I am just starting to figure it out as well and the documentation is not very good at this point of time.

I guess the best articles to start are:
http://victorsavkin.com/post/110170125256/change-detection-in-angular-2
http://victorsavkin.com/post/137821436516/managing-state-in-angular-2-applications

Give me some time and I will prepare a example for the case.

[ocombe]

Thanks, I'll look at them this week end

[hpop]

I have a hard time to get this running for an example. Do you have a working jsfiddle or plunker with ng2-translate?

In the meantime I tested #31 and it perfectly fixes my issue. If I use directives instead of pipes in my application, everything gets updated perfectly even if OnPush is used.

[sclausen]

@hpop but be aware, that you should be careful with html injection, since no sanitization will be performed.
I did a little bit of research because of @ocombe s concerns in #31, but I didn't found any lightweight solution for this problem.

[ocombe]

It look like we might need to write a sanitization library for ng2

[ocombe]

Hey @PascalPrecht any idea on how to make the pipe work with OnPush ?

[0x-r4bbit]

I don't know if there's any event system implemented in ng2-translate yet, but if there's something like a translate service, it could expose an observable which fires when the languages changes. Then the pipe could subscribe to that service and it could use the changedetector ref to mark the path til root to be checked.

[ocombe]

Thanks @PascalPrecht, it turns out that you can inject a ChangeDetectorRef in a pipe (I found out by reading the AsyncPipe source code). It should work now !

[baio]

Honestly ngxtranslate doesn't play well with OnPush change detection, for every component which uses translate pipe you has to write a lot of same boilerplate code, so think twice before use this lib if you want use it alongside with OnPush.

Has to write some helper pipe to workaround this https://gist.github.com/baio/e1da9826806ae3bad639b194958cbdcc