fork-dev / Tracker

Bug and issue tracker for Fork for Mac
505 stars 12 forks source link

[Feature request] Display invisible symbols in diff #1643

Open wcoder opened 2 years ago

wcoder commented 2 years ago

Simple changes with BOM 0xfeff.

git diff --text > diff.txt:

diff --git a/App/DataModel/UserSignup/UserAccountRequestModel.cs b/App/DataModel/UserSignup/UserAccountRequestModel.cs
index e6377184..79340483 100644
--- a/App/DataModel/UserSignup/UserAccountRequestModel.cs
+++ b/App/DataModel/UserSignup/UserAccountRequestModel.cs
@@ -1,22 +1,22 @@
-using System;
-using System.ComponentModel.DataAnnotations;
-
-namespace App.UserSignUp
-{
-   public class UserAccountRequestModel
-   {
-       public string UserGuid { get; set; }
-   }
+using System;
+using System.ComponentModel.DataAnnotations;
+
+namespace App.UserSignUp
+{
+   public class UserAccountRequestModel
+   {
+       public string UserGuid { get; set; }
+   }
 }
\ No newline at end of file

Screenshot from Sublime Text:

Screen Shot 2022-07-14 at 8 25 21 PM

Diff in ForkApp with enabled show invisible characters doesn't show anything:

Screen Shot 2022-07-14 at 8 17 26 PM

ForkApp Version 2.19.3 macOS

wcoder commented 2 years ago

The same case with carriage-return characters. Example:

diff 1.cs 2.cs | cat -t
< using System;^M
---
> using System;

where the ^M is a carriage-return character.

Diff in ForkApp with enabled show invisible characters doesn't show anything: the same as before - no diff on UI for macos/windows carriage-return characters.