ngx-text-diff icon indicating copy to clipboard operation
ngx-text-diff copied to clipboard

Published 20 hours ago • verified publisher icon ABenassi87

Angular 10 Ivy

Open Dovernh opened this issue 5 years ago • 6 comments

This likely means that the library (ngx-text-diff/lib/ngx-text-diff.module) which declares NgxTextDiffModule has not been processed correctly by ngcc, or is not compatible with Angular Ivy

Dovernh avatar Oct 28 '20 21:10 Dovernh

So i created a new Angular 10 project, copied parts of the code and i got things to work.

Dovernh avatar Oct 29 '20 16:10 Dovernh

So i created a new Angular 10 project, copied parts of the code and i got things to work.

Would you mind sharing what you did ?

gethari avatar Nov 12 '20 09:11 gethari

Hello! I am also having problems with this component when updating it to angular 10. I have disabled ivy to avoid problems and they are still incompatible Have you managed to use this component in angular 10?

Thanks

lauramgran avatar Nov 17 '20 16:11 lauramgran

So not the best solution, but it enabled to use the functionality. If you look at the code in GitHub you will see code that is 2 years old (it is the code which does the diff’ing). That code is good (synchronize scroll doesn’t work for me). The newer code which is a wrapper around the diff code has some issues I think. So what I did was removed the wrapper code and pointed my angular code at the old code angular selectors. I have made other code adjustments that are specific to my requirements. Also I made the diff code on its own module so it is no longer a npm lib. I was hoping the author would eventually fix the code so I could toss my temp solution and go back to the npm module On Nov 17, 2020, 11:32 AM -0500, lauramgran [email protected], wrote:

Hello! I am also having problems with this component when updating it to angular 10. I have disabled ivy to avoid problems and they are still incompatible Have you managed to use this component in angular 10? Thanks — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

Dovernh avatar Nov 17 '20 16:11 Dovernh

So not the best solution, but it enabled to use the functionality. If you look at the code in GitHub you will see code that is 2 years old (it is the code which does the diff’ing). That code is good (synchronize scroll doesn’t work for me). The newer code which is a wrapper around the diff code has some issues I think. So what I did was removed the wrapper code and pointed my angular code at the old code angular selectors. I have made other code adjustments that are specific to my requirements. Also I made the diff code on its own module so it is no longer a npm lib. I was hoping the author would eventually fix the code so I could toss my temp solution and go back to the npm module … On Nov 17, 2020, 11:32 AM -0500, lauramgran @.***>, wrote: Hello! I am also having problems with this component when updating it to angular 10. I have disabled ivy to avoid problems and they are still incompatible Have you managed to use this component in angular 10? Thanks — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

I did the same for angular 10

gethari avatar Nov 23 '20 12:11 gethari

Hi, I apologize for the delay, but I don't have time to update the library because I'm so busy. If anyone wants to update the library and create a pull request I would really appreciate it. Regards

ABenassi87 avatar Nov 25 '20 15:11 ABenassi87