Merge pull request #812 from Flow-Launcher/NotificationVersion

jjw24 · web-flow · commit 8f8636a9e967 · 2021-11-16T09:20:52.000+11:00
[dev] NotificationVersion
diff --git a/Flow.Launcher/Notification.cs b/Flow.Launcher/Notification.cs
@@ -11,8 +11,8 @@ internal static class Notification
         [System.Diagnostics.CodeAnalysis.SuppressMessage("Interoperability", "CA1416:Validate platform compatibility", Justification = "<Pending>")]
         public static void Show(string title, string subTitle, string iconPath)
         {
-            var legacy = Environment.OSVersion.Version.Major < 10;
-            // Handle notification for win7/8
+            var legacy = Environment.OSVersion.Version.Build < 19041;
+            // Handle notification for win7/8/early win10
             if (legacy)
             {
                 LegacyShow(title, subTitle, iconPath);

Original file line number	Diff line number	Diff line change
`@@ -11,8 +11,8 @@ internal static class Notification`
`11`	`11`	`[System.Diagnostics.CodeAnalysis.SuppressMessage("Interoperability", "CA1416:Validate platform compatibility", Justification = "<Pending>")]`
`12`	`12`	`public static void Show(string title, string subTitle, string iconPath)`
`13`	`13`	`{`
`14`		`- var legacy = Environment.OSVersion.Version.Major < 10;`
`15`		`- // Handle notification for win7/8`
	`14`	`+ var legacy = Environment.OSVersion.Version.Build < 19041;`
	`15`	`+ // Handle notification for win7/8/early win10`
`16`	`16`	`if (legacy)`
`17`	`17`	`{`
`18`	`18`	`LegacyShow(title, subTitle, iconPath);`