Feature: Migrate more dialogs

Author: BornToBeRoot

Source/NETworkManager.Utilities/VisualTreeHelper.cs

@@ -3,8 +3,25 @@
 
 namespace NETworkManager.Utilities;
 
+/// <summary>
+/// Provides helper methods for traversing and querying the visual tree of WPF elements.
+/// </summary>
+/// <remarks>The VisualTreeHelper class contains static methods that facilitate searching for and enumerating
+/// visual child elements within a WPF application's visual tree. These methods are useful for scenarios where you need
+/// to locate elements of a specific type or perform operations on all descendants of a visual element.</remarks>
 public class VisualTreeHelper
 {
+    /// <summary>
+    /// Enumerates all descendant visual children of a specified type from the visual tree starting at the given
+    /// dependency object.
+    /// </summary>
+    /// <remarks>This method performs a recursive depth-first traversal of the visual tree. It yields each
+    /// descendant of the specified type, including those nested at any depth. The enumeration is deferred and elements
+    /// are returned as they are discovered.</remarks>
+    /// <typeparam name="T">The type of visual child to search for. Must derive from DependencyObject.</typeparam>
+    /// <param name="depObj">The root of the visual tree to search. Cannot be null.</param>
+    /// <returns>An enumerable collection of all descendant elements of type T found in the visual tree. The collection is empty
+    /// if no matching elements are found.</returns>
     public static IEnumerable<T> FindVisualChildren<T>(DependencyObject depObj) where T : DependencyObject
     {
         if (depObj == null)
@@ -19,4 +36,4 @@ public static IEnumerable<T> FindVisualChildren<T>(DependencyObject depObj) wher
             foreach (var childOfChild in FindVisualChildren<T>(child)) yield return childOfChild;
         }
     }
-}
+}

Source/NETworkManager/Controls/PowerShellControl.xaml.cs

@@ -191,8 +191,8 @@ private async Task Connect()
         catch (Exception ex)
         {
             if (!_closed)
-                // ToDo: target the correct window
-                await DialogHelper.ShowMessageAsync(Application.Current.MainWindow, Strings.Error, ex.Message, ChildWindowIcon.Error);
+                // Use built-in message box because we have visual issues in the dragablz window
+                MessageBox.Show(ex.Message, Strings.Error, MessageBoxButton.OK, MessageBoxImage.Error);
         }
 
         IsConnecting = false;

Source/NETworkManager/Controls/PuTTYControl.xaml

@@ -9,7 +9,6 @@
                        xmlns:local="clr-namespace:NETworkManager.Controls"
                        xmlns:mah="http://metro.mahapps.com/winfx/xaml/controls"
                        xmlns:settings="clr-namespace:NETworkManager.Settings;assembly=NETworkManager.Settings"
-                       mah:DialogParticipation.Register="{Binding}"
                        mc:Ignorable="d" Loaded="UserControl_Loaded"
                        d:DataContext="{d:DesignInstance local:PuTTYControl}">
     <local:UserControlBase.Resources>

Source/NETworkManager/Controls/PuTTYControl.xaml.cs

@@ -5,7 +5,6 @@
 using System.Threading.Tasks;
 using System.Windows;
 using System.Windows.Input;
-using MahApps.Metro.Controls.Dialogs;
 using NETworkManager.Localization.Resources;
 using NETworkManager.Models.PuTTY;
 using NETworkManager.Settings;
@@ -30,8 +29,6 @@ private void WindowGrid_SizeChanged(object sender, SizeChangedEventArgs e)
     private bool _initialized;
     private bool _closed;
 
-    private readonly IDialogCoordinator _dialogCoordinator;
-
     private readonly Guid _tabId;
     private readonly PuTTYSessionInfo _sessionInfo;
 
@@ -77,8 +74,6 @@ public PuTTYControl(Guid tabId, PuTTYSessionInfo sessionInfo)
         InitializeComponent();
         DataContext = this;
 
-        _dialogCoordinator = DialogCoordinator.Instance;
-
         ConfigurationManager.Current.PuTTYTabCount++;
 
         _tabId = tabId;
@@ -213,17 +208,8 @@ private async Task Connect()
         catch (Exception ex)
         {
             if (!_closed)
-            {
-                var settings = AppearanceManager.MetroDialog;
-                settings.AffirmativeButtonText = Strings.OK;
-
-                ConfigurationManager.OnDialogOpen();
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.Error,
-                    ex.Message, MessageDialogStyle.Affirmative, settings);
-
-                ConfigurationManager.OnDialogClose();
-            }
+                // Use built-in message box because we have visual issues in the dragablz window
+                MessageBox.Show(ex.Message, Strings.Error, MessageBoxButton.OK, MessageBoxImage.Error);
         }
 
         IsConnecting = false;

Source/NETworkManager/Controls/TigerVNCControl.xaml

@@ -9,7 +9,6 @@
                        xmlns:local="clr-namespace:NETworkManager.Controls"
                        xmlns:mah="http://metro.mahapps.com/winfx/xaml/controls"
                        xmlns:settings="clr-namespace:NETworkManager.Settings;assembly=NETworkManager.Settings"
-                       mah:DialogParticipation.Register="{Binding}"
                        mc:Ignorable="d" Loaded="UserControl_Loaded"
                        d:DataContext="{d:DesignInstance local:TigerVNCControl}">
     <local:UserControlBase.Resources>

Source/NETworkManager/Controls/TigerVNCControl.xaml.cs

@@ -5,7 +5,6 @@
 using System.Threading.Tasks;
 using System.Windows;
 using System.Windows.Input;
-using MahApps.Metro.Controls.Dialogs;
 using NETworkManager.Localization.Resources;
 using NETworkManager.Models.TigerVNC;
 using NETworkManager.Settings;
@@ -29,8 +28,6 @@ private void TigerVNCGrid_SizeChanged(object sender, SizeChangedEventArgs e)
     private bool _initialized;
     private bool _closed;
 
-    private readonly IDialogCoordinator _dialogCoordinator;
-
     private readonly Guid _tabId;
     private readonly TigerVNCSessionInfo _sessionInfo;
 
@@ -76,8 +73,6 @@ public TigerVNCControl(Guid tabId, TigerVNCSessionInfo sessionInfo)
         InitializeComponent();
         DataContext = this;
 
-        _dialogCoordinator = DialogCoordinator.Instance;
-
         ConfigurationManager.Current.TigerVNCTabCount++;
 
         _tabId = tabId;
@@ -209,16 +204,8 @@ private async Task Connect()
         catch (Exception ex)
         {
             if (!_closed)
-            {
-                var settings = AppearanceManager.MetroDialog;
-                settings.AffirmativeButtonText = Strings.OK;
-                ConfigurationManager.OnDialogOpen();
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.Error,
-                    ex.Message, MessageDialogStyle.Affirmative, settings);
-
-                ConfigurationManager.OnDialogClose();
-            }
+                // Use built-in message box because we have visual issues in the dragablz window
+                MessageBox.Show(ex.Message, Strings.Error, MessageBoxButton.OK, MessageBoxImage.Error);
         }
 
         IsConnecting = false;

Source/NETworkManager/Controls/WebConsoleControl.xaml

@@ -10,7 +10,6 @@
                        xmlns:iconPacks="http://metro.mahapps.com/winfx/xaml/iconpacks"
                        xmlns:settings="clr-namespace:NETworkManager.Settings;assembly=NETworkManager.Settings"
                        xmlns:webview="clr-namespace:Microsoft.Web.WebView2.Wpf;assembly=Microsoft.Web.WebView2.Wpf"
-                       mah:DialogParticipation.Register="{Binding}"
                        mc:Ignorable="d" Loaded="UserControl_Loaded"
                        d:DataContext="{d:DesignInstance local:WebConsoleControl}">
     <local:UserControlBase.Resources>