Consolidate Supported property under a single class

ds5678 · ds5678 · commit bb0f63dc92b5 · 2025-07-14T01:46:42.000-07:00
diff --git a/AssetRipper.NativeDialogs.Example/Program.cs b/AssetRipper.NativeDialogs.Example/Program.cs
@@ -10,6 +10,12 @@ static async Task Main(string[] args)
 			return;
 		}
 
+		if (!NativeDialog.Supported)
+		{
+			Console.WriteLine("Native dialogs are not supported on this platform.");
+			return;
+		}
+
 		if (arguments.OpenFile)
 		{
 			if (arguments.AllowMultiple)
diff --git a/AssetRipper.NativeDialogs/ConfirmationDialog.cs b/AssetRipper.NativeDialogs/ConfirmationDialog.cs
@@ -6,11 +6,6 @@ namespace AssetRipper.NativeDialogs;
 
 public static class ConfirmationDialog
 {
-	public static bool Supported =>
-		OperatingSystem.IsWindows() ||
-		OperatingSystem.IsMacOS() ||
-		(OperatingSystem.IsLinux() && Gtk.Global.IsSupported);
-
 	public static Task<bool?> Confirm(string message, string trueLabel, string falseLabel)
 	{
 		ArgumentException.ThrowIfNullOrEmpty(message);
diff --git a/AssetRipper.NativeDialogs/MessageDialog.cs b/AssetRipper.NativeDialogs/MessageDialog.cs
@@ -6,11 +6,6 @@ namespace AssetRipper.NativeDialogs;
 
 public static class MessageDialog
 {
-	public static bool Supported =>
-		OperatingSystem.IsWindows() ||
-		OperatingSystem.IsMacOS() ||
-		(OperatingSystem.IsLinux() && Gtk.Global.IsSupported);
-
 	public static Task Message(string message)
 	{
 		if (OperatingSystem.IsWindows())
diff --git a/AssetRipper.NativeDialogs/NativeDialog.cs b/AssetRipper.NativeDialogs/NativeDialog.cs
@@ -0,0 +1,9 @@
+﻿namespace AssetRipper.NativeDialogs;
+
+public static class NativeDialog
+{
+	public static bool Supported =>
+		OperatingSystem.IsWindows() ||
+		OperatingSystem.IsMacOS() ||
+		(OperatingSystem.IsLinux() && Gtk.Global.IsSupported);
+}
diff --git a/AssetRipper.NativeDialogs/OpenFileDialog.cs b/AssetRipper.NativeDialogs/OpenFileDialog.cs
@@ -7,11 +7,6 @@ namespace AssetRipper.NativeDialogs;
 
 public static class OpenFileDialog
 {
-	public static bool Supported =>
-		OperatingSystem.IsWindows() ||
-		OperatingSystem.IsMacOS() ||
-		(OperatingSystem.IsLinux() && Gtk.Global.IsSupported);
-
 	public static Task<string?> OpenFile()
 	{
 		if (OperatingSystem.IsWindows())
diff --git a/AssetRipper.NativeDialogs/OpenFolderDialog.cs b/AssetRipper.NativeDialogs/OpenFolderDialog.cs
@@ -6,11 +6,6 @@ namespace AssetRipper.NativeDialogs;
 
 public static class OpenFolderDialog
 {
-	public static bool Supported =>
-		OperatingSystem.IsWindows() ||
-		OperatingSystem.IsMacOS() ||
-		(OperatingSystem.IsLinux() && Gtk.Global.IsSupported);
-
 	public static Task<string?> OpenFolder()
 	{
 		if (OperatingSystem.IsWindows())
diff --git a/AssetRipper.NativeDialogs/SaveFileDialog.cs b/AssetRipper.NativeDialogs/SaveFileDialog.cs
@@ -7,11 +7,6 @@ namespace AssetRipper.NativeDialogs;
 
 public static class SaveFileDialog
 {
-	public static bool Supported =>
-		OperatingSystem.IsWindows() ||
-		OperatingSystem.IsMacOS() ||
-		(OperatingSystem.IsLinux() && Gtk.Global.IsSupported);
-
 	public static Task<string?> SaveFile()
 	{
 		if (OperatingSystem.IsWindows())

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,12 @@ static async Task Main(string[] args)`
`10`	`10`	`return;`
`11`	`11`	`}`
`12`	`12`
	`13`	`+ if (!NativeDialog.Supported)`
	`14`	`+ {`
	`15`	`+ Console.WriteLine("Native dialogs are not supported on this platform.");`
	`16`	`+ return;`
	`17`	`+ }`
	`18`	`+`
`13`	`19`	`if (arguments.OpenFile)`
`14`	`20`	`{`
`15`	`21`	`if (arguments.AllowMultiple)`
Original file line number	Diff line number	Diff line change
`@@ -6,11 +6,6 @@ namespace AssetRipper.NativeDialogs;`
`6`	`6`
`7`	`7`	`public static class ConfirmationDialog`
`8`	`8`	`{`
`9`		`- public static bool Supported =>`
`10`		`- OperatingSystem.IsWindows() \|\|`
`11`		`- OperatingSystem.IsMacOS() \|\|`
`12`		`- (OperatingSystem.IsLinux() && Gtk.Global.IsSupported);`
`13`		`-`
`14`	`9`	`public static Task<bool?> Confirm(string message, string trueLabel, string falseLabel)`
`15`	`10`	`{`
`16`	`11`	`ArgumentException.ThrowIfNullOrEmpty(message);`
Original file line number	Diff line number	Diff line change
`@@ -6,11 +6,6 @@ namespace AssetRipper.NativeDialogs;`
`6`	`6`
`7`	`7`	`public static class MessageDialog`
`8`	`8`	`{`
`9`		`- public static bool Supported =>`
`10`		`- OperatingSystem.IsWindows() \|\|`
`11`		`- OperatingSystem.IsMacOS() \|\|`
`12`		`- (OperatingSystem.IsLinux() && Gtk.Global.IsSupported);`
`13`		`-`
`14`	`9`	`public static Task Message(string message)`
`15`	`10`	`{`
`16`	`11`	`if (OperatingSystem.IsWindows())`
Original file line number	Diff line number	Diff line change
`@@ -7,11 +7,6 @@ namespace AssetRipper.NativeDialogs;`
`7`	`7`
`8`	`8`	`public static class OpenFileDialog`
`9`	`9`	`{`
`10`		`- public static bool Supported =>`
`11`		`- OperatingSystem.IsWindows() \|\|`
`12`		`- OperatingSystem.IsMacOS() \|\|`
`13`		`- (OperatingSystem.IsLinux() && Gtk.Global.IsSupported);`
`14`		`-`
`15`	`10`	`public static Task<string?> OpenFile()`
`16`	`11`	`{`
`17`	`12`	`if (OperatingSystem.IsWindows())`
Original file line number	Diff line number	Diff line change
`@@ -6,11 +6,6 @@ namespace AssetRipper.NativeDialogs;`
`6`	`6`
`7`	`7`	`public static class OpenFolderDialog`
`8`	`8`	`{`
`9`		`- public static bool Supported =>`
`10`		`- OperatingSystem.IsWindows() \|\|`
`11`		`- OperatingSystem.IsMacOS() \|\|`
`12`		`- (OperatingSystem.IsLinux() && Gtk.Global.IsSupported);`
`13`		`-`
`14`	`9`	`public static Task<string?> OpenFolder()`
`15`	`10`	`{`
`16`	`11`	`if (OperatingSystem.IsWindows())`
Original file line number	Diff line number	Diff line change
`@@ -7,11 +7,6 @@ namespace AssetRipper.NativeDialogs;`
`7`	`7`
`8`	`8`	`public static class SaveFileDialog`
`9`	`9`	`{`
`10`		`- public static bool Supported =>`
`11`		`- OperatingSystem.IsWindows() \|\|`
`12`		`- OperatingSystem.IsMacOS() \|\|`
`13`		`- (OperatingSystem.IsLinux() && Gtk.Global.IsSupported);`
`14`		`-`
`15`	`10`	`public static Task<string?> SaveFile()`
`16`	`11`	`{`
`17`	`12`	`if (OperatingSystem.IsWindows())`