App Notification Samples: addressing nits from previous code-review (#219)

* Addressing nits from previous code-review

* git prefers empty lines at the end of files

Co-authored-by: Eric Langlois <[email protected]>

Author: loneursid

Samples/Notifications/App/CsUnpackagedAppNotifications/CsUnpackagedAppNotifications/MainPage.xaml.cs

@@ -63,7 +63,6 @@ public void NotificationReceived(Notification notification)
             if (DispatcherQueue.HasThreadAccess)
             {
                 messages.Insert(0, text);
-
             }
             else
             {

Samples/Notifications/App/CsUnpackagedAppNotifications/CsUnpackagedAppNotifications/Notifications/Common.cs

@@ -1,11 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-#if false
-//#include "pch.h"
-//#include "Common.h"
-#endif
-
 class Common
 {
 	const string scenarioTag = "scenarioId";
@@ -32,20 +27,20 @@ public static string ExtractParamFromArgs(string args, string paramName)
         return args.Substring(paramStart, paramEnd - paramStart);
     }
 
-	public static string MakeScenarioIdToken(int id)
-	{
-    	return scenarioTag + "=" + id.ToString();
-	}
+    public static string MakeScenarioIdToken(int id)
+    {
+        return scenarioTag + "=" + id.ToString();
+    }
 
-	public static int ExtractScenarioIdFromArgs(string args)
-	{
-    	var scenarioId = ExtractParamFromArgs(args, scenarioTag);
+    public static int ExtractScenarioIdFromArgs(string args)
+    {
+        var scenarioId = ExtractParamFromArgs(args, scenarioTag);
 
-    	if (scenarioId == null)
-    	{
-        	return 0;
-    	}
+        if (scenarioId == null)
+        {
+            return 0;
+        }
 
         return int.Parse(scenarioId);
-	}
-}
+    }
+}

Samples/Notifications/App/CsUnpackagedAppNotifications/CsUnpackagedAppNotifications/Notifications/NotificationManager.cs

@@ -1,12 +1,9 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
 
+using System;
+using System.Collections.Generic;
 using Microsoft.Windows.AppNotifications;
-using Windows.Foundation;
-using Microsoft.UI.Xaml.Controls;
 
 namespace CsUnpackagedAppNotifications.Notifications
 {
@@ -63,21 +60,21 @@ public bool DispatchNotification(AppNotificationActivatedEventArgs notificationA
         {
             var scenarioId = Common.ExtractScenarioIdFromArgs(notificationActivatedEventArgs.Argument);
             if (scenarioId != 0)
-			{
-            	try
-            	{
-                	c_notificationHandlers[scenarioId](notificationActivatedEventArgs);
-                	return true;
-            	}
-            	catch
-            	{
-            		return false; // Couldn't find a NotificationHandler for scenarioId.
-            	}
-			}
-			else
-    		{
-        		return false; // No scenario specified in the notification
-    		}
+            {
+                try
+                {
+                    c_notificationHandlers[scenarioId](notificationActivatedEventArgs);
+                    return true;
+                }
+                catch
+                {
+                    return false; // Couldn't find a NotificationHandler for scenarioId.
+                }
+            }
+            else
+            {
+                return false; // No scenario specified in the notification
+            }
         }
 
         void OnNotificationInvoked(object sender, AppNotificationActivatedEventArgs notificationActivatedEventArgs)

Samples/Notifications/App/CsUnpackagedAppNotifications/CsUnpackagedAppNotifications/Notifications/ToastWithAvatar.cs

@@ -33,8 +33,7 @@ public static bool SendToast()
         var toast = new AppNotification(xmlPayload);
         AppNotificationManager.Default.Show(toast);
 
-    	return toast.Id != 0; // return true (indicating success) if the toast was sent (if it has an Id)
-
+        return toast.Id != 0; // return true (indicating success) if the toast was sent (if it has an Id)
     }
 
     public static void NotificationReceived(AppNotificationActivatedEventArgs notificationActivatedEventArgs)

Samples/Notifications/App/CsUnpackagedAppNotifications/CsUnpackagedAppNotifications/Notifications/ToastWithTextBox.cs

@@ -44,10 +44,11 @@ public static bool SendToast()
 
     public static void NotificationReceived(AppNotificationActivatedEventArgs notificationActivatedEventArgs)
     {
-    // In a real-life scenario, this type of action would usually be processed in the background. Your App would process the payload in
-    // the background (sending the payload back to your App Server) without ever showing the App's UI.
-    // This is not something that can easily be demonstrated in a sample such as this one, as we need to show the UI to demonstrate how
-    // the payload is routed internally
+        // In a real-life scenario, this type of action would usually be processed in the background. Your App would process the payload in
+        // the background (sending the payload back to your App Server) without ever showing the App's UI.
+        // This is not something that can easily be demonstrated in a sample such as this one, as we need to show the UI to demonstrate how
+        // the payload is routed internally
+        
         var input = notificationActivatedEventArgs.UserInput;
         var text = input[textboxReplyId];