diffplex icon indicating copy to clipboard operation
diffplex copied to clipboard

Added new Diff on both DiffBuilders for more detailed DiffModel.

Open driekus77 opened this issue 3 years ago • 2 comments

See Issue #29

Added on both InlineDiffBuilder and SideBySideDiffBuilder possibility to add more details by providing a detailsPack of IChunker(s).

The InlineDiffBuilder on lowest level, is still missing the old / before char info. Didn't know where to put it: After or Under(SubPiece) the new / after char. Advice / help is welcome!

In order to not introduce any breaking changes in the API I introduced new Diff overload methods. These method(s) distinguish themselves by a detailsPack (Name oke?) that include the IChunkers to use. If null then default all 3 chunkers are used: Lines => Words => Characters.

Review comments are welcome!

Thanks for this great lib!

driekus77 avatar Jun 09 '21 09:06 driekus77

Hi guys, i am wondering why we dont merge this PR? It is very useful for inline diff to have subpieces.

ldsenow avatar Apr 03 '22 01:04 ldsenow

There are unresolved comments I believe.

mmanela avatar Apr 03 '22 01:04 mmanela