Logger cleanup

Author: tig

Terminal.Gui/ConsoleDrivers/AnsiResponseParser/AnsiMouseParser.cs

@@ -52,7 +52,7 @@ public bool IsMouse (string? cur)
                 Flags = GetFlags (buttonCode, terminator)
             };
 
-            Logging.Trace ($"{nameof (AnsiMouseParser)} handled as {input} mouse {m.Flags} at {m.Position}");
+            //Logging.Trace ($"{nameof (AnsiMouseParser)} handled as {input} mouse {m.Flags} at {m.Position}");
 
             return m;
         }

Terminal.Gui/ConsoleDrivers/V2/InputProcessor.cs

@@ -79,7 +79,7 @@ public void OnMouseEvent (MouseEventArgs a)
 
         foreach (MouseEventArgs e in _mouseInterpreter.Process (a))
         {
-            Logging.Trace ($"Mouse Interpreter raising {e.Flags}");
+           // Logging.Trace ($"Mouse Interpreter raising {e.Flags}");
 
             // Pass on
             MouseEvent?.Invoke (this, e);

UICatalog/UICatalog.cs

@@ -74,7 +74,7 @@ public class UICatalogApp
     private static Options _options;
     private static ObservableCollection<Scenario>? _scenarios;
 
-    private const string LOGFILE_LOCATION = "./logs";
+    private const string LOGFILE_LOCATION = "logs";
     private static string _logFilePath = string.Empty;
     private static readonly LoggingLevelSwitch _logLevelSwitch = new ();
 
@@ -171,7 +171,7 @@ private static int Main (string [] args)
         resultsFile.AddAlias ("--f");
 
         // what's the app name?
-        _logFilePath = $"{LOGFILE_LOCATION}/{Assembly.GetExecutingAssembly ().GetName ().Name}.log";
+        _logFilePath = $"{LOGFILE_LOCATION}/{Assembly.GetExecutingAssembly ().GetName ().Name}";
         Option<string> debugLogLevel = new Option<string> ("--debug-log-level", $"The level to use for logging (debug console and {_logFilePath})").FromAmong (
              Enum.GetNames<LogLevel> ()
             );
@@ -1354,11 +1354,14 @@ private List<MenuItem []> CreateLoggingMenuItems ()
             menuItems.Add (null!);
 
             menuItems.Add (
-                           new ()
-                           {
-                               Title = $"Log file: {_logFilePath}"
-                               //CanExecute = () => false
-                           });
+                           new (
+                                $"_Open Log Folder",
+                                "",
+                                () => OpenUrl (LOGFILE_LOCATION),
+                                null,
+                                null,
+                                null
+                               ));
 
             return menuItems.ToArray ()!;
         }

docfx/docs/logging.md

@@ -1,14 +1,20 @@
 # Logging
 
-Logging has come to Terminal.Gui! You can now enable comprehensive logging of the internals of the libray. This can help diagnose issues with specific terminals, keyboard cultures and/or operating system specific issues.
+Logging has come to Terminal.Gui! You can now enable comprehensive logging of the internals of the library. This can help diagnose issues with specific terminals, keyboard cultures and/or operating system specific issues.
 
-To enable file logging you should set the static property `Logging.Logger` to an instance of `Microsoft.Extensions.Logging.ILogger`.  If your program already uses logging you can provide a shared instance or instance from Dependency Injection (DI).
+To enable file logging you should set the static property `Logging.Logger` to an instance of `Microsoft.Extensions.Logging.ILogger`. If your program already uses logging you can provide a shared instance or instance from Dependency Injection (DI).
 
 Alternatively you can create a new log to ensure only Terminal.Gui logs appear.
 
-Any logging framework will work  (Serilog, NLog, Log4Net etc) but you should ensure you only log to File or UDP etc (i.e. not to console!).
+Any logging framework will work (Serilog, NLog, Log4Net etc) but you should ensure you don't log to the stdout console (File, Debug Output, or UDP etc... are all fine).
 
-## Worked example with Serilog to file
+## UICatalog
+
+UI Catalog has built-in UI for logging. It logs to both the debug console and a file. By default it only logs at the `Warning` level. 
+
+![UICatalog Logging](../images/UICatalog_Logging.png)
+
+## Example with Serilog to file
 
 Here is an example of how to add logging of Terminal.Gui internals to your program using Serilog file log.
 
@@ -81,7 +87,7 @@ Example logs:
 
 ## Metrics
 
-If you are finding that the UI is slow or unresponsive - or are just interested in performance metrics.  You can see these by instaling the `dotnet-counter` tool and running it for your process.
+If you are finding that the UI is slow or unresponsive - or are just interested in performance metrics. You can see these by installing the `dotnet-counter` tool and running it for your process.
 
 ```
 dotnet tool install dotnet-counters --global