From ee06bd849764c40ee8c141780629c7ca4558f4d3 Mon Sep 17 00:00:00 2001
From: Matt Parker <61717342+MattParkerDev@users.noreply.github.com>
Date: Tue, 30 Sep 2025 22:07:23 +1000
Subject: [PATCH] remove unused dialog

---
 src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.cs   |  6 +-----
 src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.tscn |  1 +
 src/SharpIDE.Godot/IdeRoot.cs                        |  4 ----
 src/SharpIDE.Godot/IdeRoot.tscn                      | 11 -----------
 4 files changed, 2 insertions(+), 20 deletions(-)

diff --git a/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.cs b/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.cs
index f48666b..dc56e4a 100644
--- a/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.cs
+++ b/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.cs
@@ -21,11 +21,7 @@ public partial class SlnPicker : Control
         _openSlnButton = GetNode<Button>("%OpenSlnButton");
         _openSlnButton.Pressed += () => _fileDialog.PopupCentered();
         var windowParent = GetParentOrNull<Window>();
-        _fileDialog.FileSelected += path =>
-        {
-            windowParent?.Hide();
-            _tcs.SetResult(path);
-        };
+        _fileDialog.FileSelected += path => _tcs.SetResult(path);
         windowParent?.CloseRequested += () => _tcs.SetResult(null);
     }
 
diff --git a/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.tscn b/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.tscn
index 58d3e3f..d599f9b 100644
--- a/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.tscn
+++ b/src/SharpIDE.Godot/Features/SlnPicker/SlnPicker.tscn
@@ -74,6 +74,7 @@ layout_mode = 2
 unique_name_in_owner = true
 oversampling_override = 1.0
 title = "Open a File"
+dialog_hide_on_ok = true
 file_mode = 0
 access = 2
 filters = PackedStringArray("*.sln,*.slnx;Solution Files")
diff --git a/src/SharpIDE.Godot/IdeRoot.cs b/src/SharpIDE.Godot/IdeRoot.cs
index 5020c4c..8c0fae9 100644
--- a/src/SharpIDE.Godot/IdeRoot.cs
+++ b/src/SharpIDE.Godot/IdeRoot.cs
@@ -19,7 +19,6 @@ public partial class IdeRoot : Control
 	public IdeWindow IdeWindow { get; set; } = null!;
 	private Button _openSlnButton = null!;
 	private Button _buildSlnButton = null!;
-	private FileDialog _fileDialog = null!;
 	private SearchWindow _searchWindow = null!;
 	private CodeEditorPanel _codeEditorPanel = null!;
 	private SolutionExplorerPanel _solutionExplorerPanel = null!;
@@ -45,7 +44,6 @@ public partial class IdeRoot : Control
 		_runMenuPopup = GetNode<Popup>("%RunMenuPopup");
 		_runMenuButton = GetNode<Button>("%RunMenuButton");
 		_codeEditorPanel = GetNode<CodeEditorPanel>("%CodeEditorPanel");
-		_fileDialog = GetNode<FileDialog>("%OpenSolutionDialog");
 		_searchWindow = GetNode<SearchWindow>("%SearchWindow");
 		_solutionExplorerPanel = GetNode<SolutionExplorerPanel>("%SolutionExplorerPanel");
 		_runPanel = GetNode<RunPanel>("%RunPanel");
@@ -54,12 +52,10 @@ public partial class IdeRoot : Control
 		
 		_runMenuButton.Pressed += OnRunMenuButtonPressed;
 		GodotGlobalEvents.Instance.FileSelected += OnSolutionExplorerPanelOnFileSelected;
-		_fileDialog.FileSelected += SetSlnFilePath;
 		_openSlnButton.Pressed += () => IdeWindow.PickSolution();
 		_buildSlnButton.Pressed += OnBuildSlnButtonPressed;
 		GodotGlobalEvents.Instance.BottomPanelVisibilityChangeRequested += async show => await this.InvokeAsync(() => _invertedVSplitContainer.InvertedSetCollapsed(!show));
 		_nodeReadyTcs.SetResult();
-		//OnSlnFileSelected(@"C:\Users\Matthew\Documents\Git\BlazorCodeBreaker\BlazorCodeBreaker.slnx");
 	}
 
 	private void OnRunMenuButtonPressed()
diff --git a/src/SharpIDE.Godot/IdeRoot.tscn b/src/SharpIDE.Godot/IdeRoot.tscn
index 7725783..9a86c29 100644
--- a/src/SharpIDE.Godot/IdeRoot.tscn
+++ b/src/SharpIDE.Godot/IdeRoot.tscn
@@ -162,17 +162,6 @@ unique_name_in_owner = true
 visible = false
 layout_mode = 1
 
-[node name="OpenSolutionDialog" type="FileDialog" parent="."]
-unique_name_in_owner = true
-title = "Open a File"
-position = Vector2i(0, 36)
-ok_button_text = "Open"
-dialog_hide_on_ok = true
-file_mode = 0
-access = 2
-filters = PackedStringArray("*.sln", "*.slnx")
-use_native_dialog = true
-
 [node name="SearchWindow" parent="." instance=ExtResource("13_7ptyn")]
 unique_name_in_owner = true
 visible = false