Im not sure what is causing the issue, but ive seen several cases where doing a copy a

The issue is with newlines - if you look at the raw output from <a href="https://githu

any idea what the solution might be? <span class="email-hidden-tog

Strange lines get inserted in the wrong place or diff computed wrong about ace-diff HOT 5 CLOSED

jackanderson commented on May 24, 2024

Strange lines get inserted in the wrong place or diff computed wrong

from ace-diff.

Comments (5)

JackuB commented on May 24, 2024

I think I know how to fix, but I’m currently travelling so I could get to it in a few days.

Possibly related to #1

from ace-diff.

jackanderson commented on May 24, 2024

hmm. nope. that didnt fix the issue. I still get a new difference after copying. It's like it's not initially detecting the blank line as a difference and then once the first difference is resolved then it detects it as a difference. Maybe something to do with the diff simplifier?

from ace-diff.

JackuB commented on May 24, 2024

Still travelling, but managed to create the smallest possible repro:

start

something else

end

and

starts
mid
end

I will be looking into why this is happening. Pushed the branch here for now https://github.com/ace-diff/ace-diff/tree/JackuB/merge-with-newlines

from ace-diff.

JackuB commented on May 24, 2024

The issue is with newlines - if you look at the raw output from https://github.com/JackuB/diff-match-patch there is \nend (0 means no change, so it's not part of the patch that's applied)

If you manually move the \n to the previous line 42b957e it would fix the diffing, as now the newline will be part of the patch.

I think it might be related to this behaviour:

ace-diff/src/index.js

Lines 494 to 497 in 42b957e

 // this was added in to hack around an oddity with the Google lib. Sometimes it would include a newline 

 // as the first char for a diff, other times not - and it would change when you were typing on-the-fly. This 

 // is used to level things out so the diffs don't appear to shift around 

 let newContentStartsWithNewline = /^\n/.test(diffText);

from ace-diff.

jackanderson commented on May 24, 2024

any idea what the solution might be?

…

On Thu, Apr 19, 2018 at 8:55 AM, Jakub Mikulas ***@***.***> wrote: The issue is with newlines - if you look at the raw output from https://github.com/JackuB/diff-match-patch there is \nend (0 means no change, so it's not part of the patch that's applied) [image: snimek obrazovky 2018-04-19 v 17 04 16] <https://user-images.githubusercontent.com/1788727/39003140-878fa57a-43fa-11e8-9e75-747c09910491.png> If you manually move the \n to the previous line 42b957e <42b957e> it would fix the diffing, as now the newline will be part of the patch. [image: 2018-04-19 17_52_12] <https://user-images.githubusercontent.com/1788727/39003244-be0efde4-43fa-11e8-89c9-31cb7f4a2ddc.gif> I think it might be related to this behaviour: https://github.com/ace-diff/ace-diff/blob/42b957ee183c614494a614d836f8c8 ed1b49d411/src/index.js#L494-L497 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#28 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AATV__HlfHRp2sUwkH9IJta7ap8rIittks5tqLN9gaJpZM4SQOWp> .

from ace-diff.

Strange lines get inserted in the wrong place or diff computed wrong about ace-diff HOT 5 CLOSED

Comments (5)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	// this was added in to hack around an oddity with the Google lib. Sometimes it would include a newline
	// as the first char for a diff, other times not - and it would change when you were typing on-the-fly. This
	// is used to level things out so the diffs don't appear to shift around
	let newContentStartsWithNewline = /^\n/.test(diffText);