search

kohsah / bungeni-portal

Automatically exported from code.google.com/p/bungeni-portal
0 stars 0 forks source link

Versions diff #757

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The diff functionality we have now doesn't work very well in some cases eg. 
addition of text in the middle of a paragraph. A simple solution is to replace 
it with a better diff algorithm,[1] works reasonably well as a drop in 
replacement. However we could improve the user friendliness of the diff 
functionality.

I suggest that we separate formatting changes from content changes as DaisyDiff 
does[3]. Content changes ie. insertions and deletions, will continue to be 
handled in the same way as they are now. Formatting changes eg. bold, italics, 
font changes etc will be marked somehow eg. with a zigzag underline. The 
formatting changes will then be displayed when a user mouse overs over the text 
in an overlay.

In the current code a formatting change is shown as a deletion then insertion 
which in reality is not what is happening.

[1] https://github.com/cygri/htmldiff
[2] http://code.google.com/p/daisydiff/wiki/DaisyDiffModes

Original issue reported on code.google.com by mianonj...@gmail.com on 12 Apr 2012 at 12:49

GoogleCodeExporter commented 8 years ago 
This issue was fixed with a better diffing implementation

Original comment by ashok.ha...@gmail.com on 3 Dec 2012 at 12:32