Just paint the foreground when colouring a log message.

For modern terminals that are on black, there's no difference. For ISE users with a blue background, it's nicer to not colour it. The ISE doesn't, so why should we?

Author: mjr4077au

src/PSADT/PSAppDeployToolkit/Logging/LogUtilities.cs

@@ -143,7 +143,6 @@ public static IReadOnlyList<LogEntry> WriteLogEntry(IReadOnlyList<string> messag
                     if (colouredOutput)
                     {
                         Console.ForegroundColor = sevCols["ForegroundColor"];
-                        Console.BackgroundColor = sevCols["BackgroundColor"];
                     }
                     if (severity == LogSeverity.Error)
                     {
@@ -296,10 +295,10 @@ static void AppendLowMarker(StringBuilder sb, char low)
         /// </summary>
         private static readonly ReadOnlyCollection<ReadOnlyDictionary<string, ConsoleColor>> LogSeverityColors = new(
         [
-            new(new Dictionary<string, ConsoleColor>() { { "ForegroundColor", ConsoleColor.Green }, { "BackgroundColor", ConsoleColor.Black } }),
+            new(new Dictionary<string, ConsoleColor>() { { "ForegroundColor", ConsoleColor.Green } }),
             new(new Dictionary<string, ConsoleColor>() { }),
-            new(new Dictionary<string, ConsoleColor>() { { "ForegroundColor", ConsoleColor.Yellow }, { "BackgroundColor", ConsoleColor.Black } }),
-            new(new Dictionary<string, ConsoleColor>() { { "ForegroundColor", ConsoleColor.Red }, { "BackgroundColor", ConsoleColor.Black } }),
+            new(new Dictionary<string, ConsoleColor>() { { "ForegroundColor", ConsoleColor.Yellow } }),
+            new(new Dictionary<string, ConsoleColor>() { { "ForegroundColor", ConsoleColor.Red } }),
         ]);
 
         /// <summary>