-
-
Notifications
You must be signed in to change notification settings - Fork 189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cannot diff spaces #94
Comments
It does support that. Let me show you some sample code. The below code will take the example you gave and print out two lines, the old and the new while highlighting any changes. I replace spaces with dots to make it easier to see. Here is the code: private static void Main()
{
var diffBuilder = new SideBySideDiffBuilder(new Differ());
var diff = diffBuilder.BuildDiffModel("This is text 1", "This is text 1", ignoreWhitespace: false);
PrintDiffSide(diff.OldText);
PrintDiffSide(diff.NewText);
}
private static void PrintDiffSide(DiffPaneModel diff)
{
foreach (var line in diff.Lines)
{
foreach (var part in line.SubPieces)
{
var partWithSpacesMadeVisible = part.Text.Replace(" ", "·");
switch (part.Type)
{
case ChangeType.Inserted:
Console.ForegroundColor = ConsoleColor.Green;
break;
case ChangeType.Deleted:
Console.ForegroundColor = ConsoleColor.Red;
break;
case ChangeType.Unchanged:
Console.ForegroundColor = ConsoleColor.White;
break;
}
Console.Write(partWithSpacesMadeVisible);
}
}
Console.WriteLine();
} @lchthanhth615 let me know if you have any questions on this |
Thanks @mmanela for your response. It will work if we replace space to Dot like your example, but it's not make sense in my case. Could you please help to check ? |
Most likely the WPF control is not exposing the setting to consider white space |
"This is text 1" and "this is text [many spaces here] 1" => it can not show diff for spaces.
The text was updated successfully, but these errors were encountered: