ABenassi87 / ngx-text-diff

A Text Diff component for Angular
https://ngx-text-diff.herokuapp.com/home
50 stars 44 forks source link

Compare result may change lines ordering? #60

Open ericxbenoit opened 3 years ago

ericxbenoit commented 3 years ago

Hello,

We are using ngx-text-diff in an Angular app and we encountered an issue with the order of the comparison results.

For example, if we have these 5 lines in the left content:

- name: DDD
- name: CCC
- name: BBB
- name: AAA
- name: EEE

And these 5 lines in the right content:

- name: EEE
- name: DDD
- name: AAA
- name: BBB
- name: CCC

The compare result in the right side is:

- name: EEE
- name: DDD
- name: AAA
- name: CCC
- name: BBB

For some unknown reason it inverts the lines BBB and CCC..

Is there a way to fix this problem quickly?

Thanks

ericxbenoit commented 3 years ago

More visual example: NgxTextDiffApp

jonathan-fortin commented 3 years ago

Hi,

I have the same problem. Anyone have a suggestion or a quick solution to this problem?

Thanks

ABenassi87 commented 3 years ago

Hi, @ericxbenoit and @jonathan-fortin thank you so much for creating the issue. I will try to review it soon and I will push the fix.

ericxbenoit commented 3 years ago

@ABenassi87 thanks, your help is really appreciated and I will wait for your fix. Thank you

jonathan-fortin commented 3 years ago

Wow, quick reply, thanks @ABenassi87 !

jonathan-fortin commented 3 years ago

Hi @ABenassi87 do you have any update on this issue ? Do you think it will be possible to have a fixed soon? Thank you!

ABenassi87 commented 3 years ago

Hi @jonathan-fortin I'm still working on it. Right now, I hope to have the fix soon.

jonathan-fortin commented 3 years ago

@ABenassi87 Thanks for the update!

ericxbenoit commented 3 years ago

Hello, @ABenassi87 Any update with the fix? Thanks

jonathan-fortin commented 3 years ago

Hi @ABenassi87

Your library is amazing, easy to integrate, good UX etc. But, this bug is a blocker in our case. Do you think it will be possible to solve it?

And thank you again for your work!

jonathan-fortin commented 3 years ago

Hi @ABenassi87,

Hope you are feeling well!

I noticed that you have branched out to resolve this issue. Do you have any idea when you will be releasing a new version with this fix?

Thanks