From e5ea852c3bce6634b11927998eb7f82b647a892a Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Sat, 8 Nov 2025 23:56:41 +0100
Subject: [PATCH 01/10] Feature: Migrate password dialogs to childwindow

---
 Source/NETworkManager/DialogHelper.cs         | 36 +++++++++++++
 .../ViewModels/ProfileFileViewModel.cs        |  6 +--
 .../ViewModels/SettingsProfilesViewModel.cs   | 52 +++++++++----------
 .../CredentialsChangePasswordDialog.xaml      |  2 +-
 .../CredentialsChangePasswordDialog.xaml.cs   | 11 ++--
 ...ml => CredentialsPasswordChildWindow.xaml} | 21 ++++----
 .../CredentialsPasswordChildWindow.xaml.cs    | 21 ++++++++
 .../Views/CredentialsPasswordDialog.xaml.cs   | 16 ------
 .../Views/ProfileFileChildWindow.xaml         |  2 +-
 9 files changed, 108 insertions(+), 59 deletions(-)
 create mode 100644 Source/NETworkManager/DialogHelper.cs
 rename Source/NETworkManager/Views/{CredentialsPasswordDialog.xaml => CredentialsPasswordChildWindow.xaml} (77%)
 create mode 100644 Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml.cs
 delete mode 100644 Source/NETworkManager/Views/CredentialsPasswordDialog.xaml.cs

diff --git a/Source/NETworkManager/DialogHelper.cs b/Source/NETworkManager/DialogHelper.cs
new file mode 100644
index 0000000000..83062f7dad
--- /dev/null
+++ b/Source/NETworkManager/DialogHelper.cs
@@ -0,0 +1,36 @@
+using MahApps.Metro.SimpleChildWindow;
+using NETworkManager.Localization.Resources;
+using NETworkManager.Settings;
+using NETworkManager.Utilities;
+using NETworkManager.ViewModels;
+using NETworkManager.Views;
+using System.Threading.Tasks;
+using System.Windows;
+
+namespace NETworkManager
+{
+    public static class DialogHelper
+    {
+        public static Task ShowOKMessageAsync(Window parentWindow, string title, string message, ChildWindowIcon icon = ChildWindowIcon.Info, string buttonOK = null)
+        {
+            if (string.IsNullOrEmpty(buttonOK))
+                buttonOK = Strings.OK;
+
+            var childWindow = new OKMessageChildWindow();
+
+            var childWindowViewModel = new OKMessageViewModel(_ =>
+            {
+                childWindow.IsOpen = false;
+                ConfigurationManager.Current.IsChildWindowOpen = false;
+            }, message, buttonOK, icon);
+
+            childWindow.Title = title;
+
+            childWindow.DataContext = childWindowViewModel;
+
+            ConfigurationManager.Current.IsChildWindowOpen = true;
+
+            return parentWindow.ShowChildWindowAsync(childWindow);
+        }
+    }
+}
diff --git a/Source/NETworkManager/ViewModels/ProfileFileViewModel.cs b/Source/NETworkManager/ViewModels/ProfileFileViewModel.cs
index 0474c56cab..f5a9c24ae3 100644
--- a/Source/NETworkManager/ViewModels/ProfileFileViewModel.cs
+++ b/Source/NETworkManager/ViewModels/ProfileFileViewModel.cs
@@ -11,10 +11,10 @@ public class ProfileFileViewModel : ViewModelBase
 
     private string _name;
 
-    public ProfileFileViewModel(Action<ProfileFileViewModel> addCommand, Action<ProfileFileViewModel> cancelHandler,
+    public ProfileFileViewModel(Action<ProfileFileViewModel> okCommand, Action<ProfileFileViewModel> cancelHandler,
         ProfileFileInfo info = null)
     {
-        AcceptCommand = new RelayCommand(_ => addCommand(this));
+        OKCommand = new RelayCommand(_ => okCommand(this));
         CancelCommand = new RelayCommand(_ => cancelHandler(this));
 
         if (info == null)
@@ -25,7 +25,7 @@ public ProfileFileViewModel(Action<ProfileFileViewModel> addCommand, Action<Prof
         IsEdit = true;
     }
 
-    public ICommand AcceptCommand { get; }
+    public ICommand OKCommand { get; }
 
     public ICommand CancelCommand { get; }
 
diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index 1290260821..78e507d865 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -213,7 +213,7 @@ private async Task EditProfileFileAction()
     }
 
     public ICommand DeleteProfileFileCommand =>
-        new RelayCommand(_ => DeleteProfileFileAction().ConfigureAwait(false), DeleteProfileFile_CanExecute);
+        new RelayCommand(async _ => await DeleteProfileFileAction().ConfigureAwait(false), DeleteProfileFile_CanExecute);
 
     private bool DeleteProfileFile_CanExecute(object obj)
     {
@@ -361,18 +361,16 @@ await _dialogCoordinator.ShowMessageAsync(this, Strings.DecryptionError,
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    public ICommand DisableEncryptionCommand => new RelayCommand(_ => DisableEncryptionAction());
+    public ICommand DisableEncryptionCommand => new RelayCommand(async _ => await DisableEncryptionAction().ConfigureAwait(false));
 
-    private async void DisableEncryptionAction()
+    private Task DisableEncryptionAction()
     {
-        var customDialog = new CustomDialog
-        {
-            Title = Strings.MasterPassword
-        };
+        var childWindow = new CredentialsPasswordChildWindow();
 
-        var credentialsPasswordViewModel = new CredentialsPasswordViewModel(async instance =>
+        var childWindowViewModel = new CredentialsPasswordViewModel(async instance =>
         {
-            await _dialogCoordinator.HideMetroDialogAsync(this, customDialog);
+            childWindow.IsOpen = false;
+            ConfigurationManager.Current.IsChildWindowOpen = false;
 
             try
             {
@@ -380,31 +378,33 @@ private async void DisableEncryptionAction()
             }
             catch (CryptographicException)
             {
-                var settings = AppearanceManager.MetroDialog;
-                settings.AffirmativeButtonText = Strings.OK;
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.WrongPassword,
-                    Strings.WrongPasswordDecryptionFailedMessage, MessageDialogStyle.Affirmative,
-                    settings);
+                await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
+                    Strings.WrongPassword,
+                    Strings.WrongPasswordDecryptionFailedMessage,
+                    ChildWindowIcon.Error).ConfigureAwait(false);
             }
             catch (Exception ex)
             {
-                var settings = AppearanceManager.MetroDialog;
-                settings.AffirmativeButtonText = Strings.OK;
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.DecryptionError,
+                await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
+                    Strings.DecryptionError,
                     $"{Strings.DecryptionErrorMessage}\n\n{ex.Message}",
-                    MessageDialogStyle.Affirmative, settings);
+                    ChildWindowIcon.Error).ConfigureAwait(false);
             }
-        }, async _1 => { await _dialogCoordinator.HideMetroDialogAsync(this, customDialog); });
 
-        customDialog.Content = new CredentialsPasswordDialog
+            }, _ =>
         {
-            DataContext = credentialsPasswordViewModel
-        };
+            childWindow.IsOpen = false;
+            ConfigurationManager.Current.IsChildWindowOpen = false;
+        });
 
-        await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
-    }
+        childWindow.Title = Strings.MasterPassword;
+
+        childWindow.DataContext = childWindowViewModel;
+
+        ConfigurationManager.Current.IsChildWindowOpen = true;
+
+        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+    }    
 
     #endregion
 }
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml b/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml
index 5e176c0cc4..57ab8188d1 100644
--- a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml
+++ b/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml
@@ -7,7 +7,7 @@
              xmlns:interactivity="http://schemas.microsoft.com/xaml/behaviors"
              xmlns:viewModels="clr-namespace:NETworkManager.ViewModels"
              xmlns:localization="clr-namespace:NETworkManager.Localization.Resources;assembly=NETworkManager.Localization"
-             mc:Ignorable="d" Loaded="UserControl_Loaded"
+             mc:Ignorable="d" Loaded="ChildWindow_OnLoaded"
              d:DataContext="{d:DesignInstance viewModels:CredentialsChangePasswordViewModel}">
     <Grid Margin="0,20">
         <Grid.RowDefinitions>
diff --git a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs b/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs
index 088f45d549..f9bb4c192c 100644
--- a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs
+++ b/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs
@@ -1,4 +1,6 @@
-using System.Windows;
+using System;
+using System.Windows;
+using System.Windows.Threading;
 
 namespace NETworkManager.Views;
 
@@ -9,8 +11,11 @@ public CredentialsChangePasswordDialog()
         InitializeComponent();
     }
 
-    private void UserControl_Loaded(object sender, RoutedEventArgs e)
+    private void ChildWindow_OnLoaded(object sender, RoutedEventArgs e)
     {
-        PasswordBoxPassword.Focus();
+        Dispatcher.BeginInvoke(DispatcherPriority.ContextIdle, new Action(delegate
+        {
+            PasswordBoxPassword.Focus();
+        }));
     }
 }
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsPasswordDialog.xaml b/Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml
similarity index 77%
rename from Source/NETworkManager/Views/CredentialsPasswordDialog.xaml
rename to Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml
index 045a3e74b3..73f66d1797 100644
--- a/Source/NETworkManager/Views/CredentialsPasswordDialog.xaml
+++ b/Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml
@@ -1,4 +1,4 @@
-<UserControl x:Class="NETworkManager.Views.CredentialsPasswordDialog"
+<simpleChildWindow:ChildWindow x:Class="NETworkManager.Views.CredentialsPasswordChildWindow"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
@@ -7,9 +7,12 @@
              xmlns:interactivity="http://schemas.microsoft.com/xaml/behaviors"
              xmlns:viewModels="clr-namespace:NETworkManager.ViewModels"
              xmlns:localization="clr-namespace:NETworkManager.Localization.Resources;assembly=NETworkManager.Localization"
-             mc:Ignorable="d" Loaded="UserControl_Loaded"
-             d:DataContext="{d:DesignInstance viewModels:CredentialsPasswordViewModel}">
-    <Grid Margin="0,20">
+             xmlns:simpleChildWindow="clr-namespace:MahApps.Metro.SimpleChildWindow;assembly=MahApps.Metro.SimpleChildWindow"             
+             CloseButtonCommand="{Binding Path=CancelCommand}"
+             Style="{StaticResource DefaultChildWindow}"
+             Loaded="ChildWindow_OnLoaded"                            
+             mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:CredentialsPasswordViewModel}">    
+    <Grid Margin="10">
         <Grid.RowDefinitions>
             <RowDefinition Height="*" />
             <RowDefinition Height="10" />
@@ -30,16 +33,16 @@
             </Grid.ColumnDefinitions>
             <TextBlock Grid.Column="0" Grid.Row="0" Text="{x:Static localization:Strings.Password}" />
             <PasswordBox x:Name="PasswordBoxPassword" Grid.Column="2" Grid.Row="0"
-                         Style="{StaticResource DefaultPasswordBox}">
+                 Style="{StaticResource DefaultPasswordBox}">
                 <interactivity:Interaction.Behaviors>
                     <wpfHelpers:PasswordBoxBindingBehavior
-                        Password="{Binding Password, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
+                Password="{Binding Password, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
                 </interactivity:Interaction.Behaviors>
             </PasswordBox>
         </Grid>
         <StackPanel Grid.Row="2" Orientation="Horizontal" HorizontalAlignment="Right">
             <Button Content="{x:Static localization:Strings.OK}" Command="{Binding OKCommand}" IsDefault="True"
-                    Margin="0,0,10,0">
+            Margin="0,0,10,0">
                 <Button.Style>
                     <Style TargetType="{x:Type Button}" BasedOn="{StaticResource HighlightedButton}">
                         <Setter Property="IsEnabled" Value="False" />
@@ -52,7 +55,7 @@
                 </Button.Style>
             </Button>
             <Button Content="{x:Static localization:Strings.Cancel}" Command="{Binding CancelCommand}" IsCancel="True"
-                    Style="{StaticResource DefaultButton}" />
+            Style="{StaticResource DefaultButton}" />
         </StackPanel>
     </Grid>
-</UserControl>
\ No newline at end of file
+</simpleChildWindow:ChildWindow>
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml.cs b/Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml.cs
new file mode 100644
index 0000000000..4ff3cc09f4
--- /dev/null
+++ b/Source/NETworkManager/Views/CredentialsPasswordChildWindow.xaml.cs
@@ -0,0 +1,21 @@
+using System;
+using System.Windows;
+using System.Windows.Threading;
+
+namespace NETworkManager.Views;
+
+public partial class CredentialsPasswordChildWindow
+{
+    public CredentialsPasswordChildWindow()
+    {
+        InitializeComponent();
+    }
+
+    private void ChildWindow_OnLoaded(object sender, RoutedEventArgs e)
+    {
+        Dispatcher.BeginInvoke(DispatcherPriority.ContextIdle, new Action(delegate
+        {
+            PasswordBoxPassword.Focus();
+        }));
+    }
+}
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsPasswordDialog.xaml.cs b/Source/NETworkManager/Views/CredentialsPasswordDialog.xaml.cs
deleted file mode 100644
index 0315f23eda..0000000000
--- a/Source/NETworkManager/Views/CredentialsPasswordDialog.xaml.cs
+++ /dev/null
@@ -1,16 +0,0 @@
-using System.Windows;
-
-namespace NETworkManager.Views;
-
-public partial class CredentialsPasswordDialog
-{
-    public CredentialsPasswordDialog()
-    {
-        InitializeComponent();
-    }
-
-    private void UserControl_Loaded(object sender, RoutedEventArgs e)
-    {
-        PasswordBoxPassword.Focus();
-    }
-}
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/ProfileFileChildWindow.xaml b/Source/NETworkManager/Views/ProfileFileChildWindow.xaml
index d990e6dc9c..4095e9f17b 100644
--- a/Source/NETworkManager/Views/ProfileFileChildWindow.xaml
+++ b/Source/NETworkManager/Views/ProfileFileChildWindow.xaml
@@ -47,7 +47,7 @@
             </TextBox>
         </Grid>
         <StackPanel Grid.Row="2" Orientation="Horizontal" HorizontalAlignment="Right">
-            <Button Command="{Binding AcceptCommand}" IsDefault="True" Margin="0,0,10,0">
+            <Button Command="{Binding OKCommand}" IsDefault="True" Margin="0,0,10,0">
                 <Button.Style>
                     <Style TargetType="{x:Type Button}" BasedOn="{StaticResource HighlightedButton}">
                         <Setter Property="IsEnabled" Value="False" />

From c0d4c6262fea144d1059acf6acf59470db449654 Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Sun, 9 Nov 2025 00:15:35 +0100
Subject: [PATCH 02/10] Feature: Migrate password dialogs to childwindow / code
 cleanup

---
 Source/NETworkManager/DialogHelper.cs         | 70 +++++++++++++++++--
 Source/NETworkManager/NETworkManager.csproj   |  2 +-
 Source/NETworkManager/ProfileDialogManager.cs | 11 +--
 .../AWSSessionManagerSettingsViewModel.cs     |  5 +-
 .../ViewModels/DNSLookupSettingsViewModel.cs  |  5 +-
 .../ViewModels/HostsFileEditorViewModel.cs    |  9 +--
 .../ViewModels/IPScannerSettingsViewModel.cs  |  5 +-
 ...ewModel.cs => OKCancelMessageViewModel.cs} |  6 +-
 .../ViewModels/OKMessageViewModel.cs          |  2 +-
 .../PortScannerSettingsViewModel.cs           |  6 +-
 .../ViewModels/SNMPSettingsViewModel.cs       |  6 +-
 .../ViewModels/SNTPLookupSettingsViewModel.cs |  6 +-
 .../ViewModels/SettingsProfilesViewModel.cs   | 59 +++++-----------
 .../ViewModels/SettingsSettingsViewModel.cs   |  8 +--
 .../ViewModels/WebConsoleSettingsViewModel.cs |  5 +-
 ...w.xaml => OKCancelMessageChildWindow.xaml} |  4 +-
 ....cs => OKCancelMessageChildWindow.xaml.cs} |  4 +-
 17 files changed, 128 insertions(+), 85 deletions(-)
 rename Source/NETworkManager/ViewModels/{OKCancelInfoMessageViewModel.cs => OKCancelMessageViewModel.cs} (83%)
 rename Source/NETworkManager/Views/{OKCancelInfoMessageChildWindow.xaml => OKCancelMessageChildWindow.xaml} (97%)
 rename Source/NETworkManager/Views/{OKCancelInfoMessageChildWindow.xaml.cs => OKCancelMessageChildWindow.xaml.cs} (79%)

diff --git a/Source/NETworkManager/DialogHelper.cs b/Source/NETworkManager/DialogHelper.cs
index 83062f7dad..7fefe0fe5b 100644
--- a/Source/NETworkManager/DialogHelper.cs
+++ b/Source/NETworkManager/DialogHelper.cs
@@ -9,12 +9,26 @@
 
 namespace NETworkManager
 {
+    /// <summary>
+    /// Helper class for showing dialog messages.
+    /// </summary>
     public static class DialogHelper
     {
-        public static Task ShowOKMessageAsync(Window parentWindow, string title, string message, ChildWindowIcon icon = ChildWindowIcon.Info, string buttonOK = null)
+        /// <summary>
+        /// Displays a modal message dialog with an OK button, allowing the user to acknowledge the message before
+        /// continuing.
+        /// </summary>
+        /// <remarks>The dialog is shown as a child window of the specified parent. The method is
+        /// asynchronous and returns when the dialog is dismissed by the user.</remarks>
+        /// <param name="parentWindow">The parent window that will host the message dialog. Cannot be null.</param>
+        /// <param name="title">The title text displayed in the message dialog window. Cannot be null.</param>
+        /// <param name="message">The main message content shown in the dialog. Cannot be null.</param>
+        /// <param name="icon">The icon to display in the dialog, indicating the message type. Defaults to Info if not specified.</param>
+        /// <param name="buttonOKText">The text to display on the OK button. If null, a default value is used.</param>
+        /// <returns>A task that completes when the user closes the message dialog.</returns>
+        public static Task ShowOKMessageAsync(Window parentWindow, string title, string message, ChildWindowIcon icon = ChildWindowIcon.Info, string buttonOKText = null)
         {
-            if (string.IsNullOrEmpty(buttonOK))
-                buttonOK = Strings.OK;
+            buttonOKText ??= Strings.OK;
 
             var childWindow = new OKMessageChildWindow();
 
@@ -22,7 +36,7 @@ public static Task ShowOKMessageAsync(Window parentWindow, string title, string
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
-            }, message, buttonOK, icon);
+            }, message, icon, buttonOKText);
 
             childWindow.Title = title;
 
@@ -32,5 +46,53 @@ public static Task ShowOKMessageAsync(Window parentWindow, string title, string
 
             return parentWindow.ShowChildWindowAsync(childWindow);
         }
+
+        /// <summary>
+        /// Displays an asynchronous modal dialog with OK and Cancel buttons, allowing the user to confirm or cancel an
+        /// action.
+        /// </summary>
+        /// <remarks>The dialog is shown as a child window of the specified parent. The method does not
+        /// return until the user closes the dialog. Custom button text and icon can be provided to tailor the dialog to
+        /// specific scenarios.</remarks>
+        /// <param name="parentWindow">The parent window that hosts the child dialog. Cannot be null.</param>
+        /// <param name="title">The title text displayed in the dialog window.</param>
+        /// <param name="message">The message content shown to the user in the dialog.</param>
+        /// <param name="icon">The icon displayed in the dialog to indicate the message type. Defaults to Info.</param>
+        /// <param name="buttonOKText">The text label for the OK button. If null, a default value is used.</param>
+        /// <param name="buttonCancelText">The text label for the Cancel button. If null, a default value is used.</param>
+        /// <returns>A task that represents the asynchronous operation. The task result is <see langword="true"/> if the user
+        /// clicks OK; otherwise, <see langword="false"/>.</returns>
+        public static async Task<bool> ShowOKCancelMessageAsync(Window parentWindow, string title, string message, ChildWindowIcon icon = ChildWindowIcon.Info, string buttonOKText = null, string buttonCancelText = null)
+        {
+            buttonOKText ??= Strings.OK;            
+            buttonCancelText ??= Strings.Cancel;
+
+            var result = false;
+
+            var childWindow = new OKCancelMessageChildWindow();
+
+            var childWindowViewModel = new OKCancelMessageViewModel(_ =>
+            {
+                childWindow.IsOpen = false;
+                ConfigurationManager.Current.IsChildWindowOpen = false;
+
+                result = true;
+            },
+            _ =>
+            {
+                childWindow.IsOpen = false;
+                ConfigurationManager.Current.IsChildWindowOpen = false;
+            },
+            message, icon, buttonOKText, buttonCancelText);
+            
+            childWindow.Title = title;
+            childWindow.DataContext = childWindowViewModel;
+            
+            ConfigurationManager.Current.IsChildWindowOpen = true;
+
+            await parentWindow.ShowChildWindowAsync(childWindow);
+
+            return result;
+        }
     }
 }
diff --git a/Source/NETworkManager/NETworkManager.csproj b/Source/NETworkManager/NETworkManager.csproj
index 6a5b70c46c..6bfdefad2d 100644
--- a/Source/NETworkManager/NETworkManager.csproj
+++ b/Source/NETworkManager/NETworkManager.csproj
@@ -141,7 +141,7 @@
           <XamlRuntime>Wpf</XamlRuntime>
           <SubType>Designer</SubType>
         </Page>
-        <Page Update="Views\OKCancelInfoMessageChildWindow.xaml">
+        <Page Update="Views\OKCancelMessageChildWindow.xaml">
           <Generator>MSBuild:Compile</Generator>
           <XamlRuntime>Wpf</XamlRuntime>
           <SubType>Designer</SubType>
diff --git a/Source/NETworkManager/ProfileDialogManager.cs b/Source/NETworkManager/ProfileDialogManager.cs
index 5766dc423b..5c17ff6c95 100644
--- a/Source/NETworkManager/ProfileDialogManager.cs
+++ b/Source/NETworkManager/ProfileDialogManager.cs
@@ -6,6 +6,7 @@
 using NETworkManager.Models.PuTTY;
 using NETworkManager.Models.RemoteDesktop;
 using NETworkManager.Profiles;
+using NETworkManager.Utilities;
 using NETworkManager.ViewModels;
 using NETworkManager.Views;
 using System;
@@ -563,9 +564,9 @@ public static Task ShowCopyAsProfileDialog(Window parentWindow, IProfileManagerM
     public static Task ShowDeleteProfileDialog(Window parentWindow, IProfileManagerMinimal viewModel,
         IList<ProfileInfo> profiles)
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        OKCancelInfoMessageViewModel childWindowViewModel = new(_ =>
+        OKCancelMessageViewModel childWindowViewModel = new(_ =>
             {
                 childWindow.IsOpen = false;
                 Settings.ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -581,6 +582,7 @@ public static Task ShowDeleteProfileDialog(Window parentWindow, IProfileManagerM
                 viewModel.OnProfileManagerDialogClose();
             },
                 profiles.Count == 1 ? Strings.DeleteProfileMessage : Strings.DeleteProfilesMessage,
+                Utilities.ChildWindowIcon.Info,
                 Strings.Delete
             );
 
@@ -665,9 +667,9 @@ public static Task ShowEditGroupDialog(Window parentWindow, IProfileManagerMinim
     public static Task ShowDeleteGroupDialog(Window parentWindow, IProfileManagerMinimal viewModel,
         GroupInfo group)
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        OKCancelInfoMessageViewModel childWindowViewModel = new(_ =>
+        OKCancelMessageViewModel childWindowViewModel = new(_ =>
         {
             childWindow.IsOpen = false;
             Settings.ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -683,6 +685,7 @@ public static Task ShowDeleteGroupDialog(Window parentWindow, IProfileManagerMin
             viewModel.OnProfileManagerDialogClose();
         },
             Strings.DeleteGroupMessage,
+            ChildWindowIcon.Info, 
             Strings.Delete
         );
 
diff --git a/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs
index 8b87e4831d..87b942eb2c 100644
--- a/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs
@@ -277,9 +277,9 @@ public async Task EditAWSProfile()
 
     private Task DeleteAWSProfile()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -291,6 +291,7 @@ private Task DeleteAWSProfile()
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
                 Strings.DeleteAWSProfileMessage,
+                ChildWindowIcon.Info,
                 Strings.Delete
             );
 
diff --git a/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs b/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs
index 40ae51f3a6..5dc6f03428 100644
--- a/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs
@@ -359,9 +359,9 @@ public async Task EditDNSServer()
 
     private Task DeleteDNSServer()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -373,6 +373,7 @@ private Task DeleteDNSServer()
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
                 Strings.DeleteDNSServerMessage,
+                ChildWindowIcon.Info,
                 Strings.Delete
             );
 
diff --git a/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs b/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
index 847df2f759..85881e987e 100644
--- a/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
+++ b/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
@@ -391,9 +391,9 @@ private async Task DeleteEntryAction()
     {
         IsModifying = true;
 
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(async _ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(async _ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -413,6 +413,7 @@ private async Task DeleteEntryAction()
             },
             string.Format(Strings.DeleteHostsFileEntryMessage, SelectedResult.IPAddress, SelectedResult.Hostname,
                 string.IsNullOrEmpty(SelectedResult.Comment) ? "" : $"# {SelectedResult.Comment}"),
+            ChildWindowIcon.Info,
             Strings.Delete
         );
 
@@ -452,7 +453,7 @@ private async Task ShowErrorMessageAsync(HostsFileEntryModifyResult result)
         {
             childWindow.IsOpen = false;
             ConfigurationManager.Current.IsChildWindowOpen = false;
-        }, message, Strings.OK, ChildWindowIcon.Error);
+        }, message, ChildWindowIcon.Error, Strings.OK);
 
         childWindow.Title = Strings.Error;
 
@@ -479,7 +480,7 @@ private async Task RestartAsAdminAction()
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
-            }, ex.Message, Strings.OK, ChildWindowIcon.Error);
+            }, ex.Message, ChildWindowIcon.Error, Strings.OK);
 
             childWindow.Title = Strings.Error;
 
diff --git a/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs
index 055028e2d4..35cb90c59c 100644
--- a/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs
@@ -387,9 +387,9 @@ public async void EditCustomCommand()
 
     private Task DeleteCustomCommand()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -401,6 +401,7 @@ private Task DeleteCustomCommand()
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
                 Strings.DeleteCustomCommandMessage,
+                ChildWindowIcon.Info,
                 Strings.Delete
             );
 
diff --git a/Source/NETworkManager/ViewModels/OKCancelInfoMessageViewModel.cs b/Source/NETworkManager/ViewModels/OKCancelMessageViewModel.cs
similarity index 83%
rename from Source/NETworkManager/ViewModels/OKCancelInfoMessageViewModel.cs
rename to Source/NETworkManager/ViewModels/OKCancelMessageViewModel.cs
index 69f4f83266..63f29342b8 100644
--- a/Source/NETworkManager/ViewModels/OKCancelInfoMessageViewModel.cs
+++ b/Source/NETworkManager/ViewModels/OKCancelMessageViewModel.cs
@@ -4,10 +4,10 @@
 
 namespace NETworkManager.ViewModels;
 
-public class OKCancelInfoMessageViewModel : ViewModelBase
+public class OKCancelMessageViewModel : ViewModelBase
 {
-    public OKCancelInfoMessageViewModel(Action<OKCancelInfoMessageViewModel> okCommand,
-        Action<OKCancelInfoMessageViewModel> cancelHandler, string message, string okButtonText = null, string cancelButtonText = null, ChildWindowIcon icon = ChildWindowIcon.Info)
+    public OKCancelMessageViewModel(Action<OKCancelMessageViewModel> okCommand,
+        Action<OKCancelMessageViewModel> cancelHandler, string message, ChildWindowIcon icon = ChildWindowIcon.Info, string okButtonText = null, string cancelButtonText = null)
     {
         OKCommand = new RelayCommand(_ => okCommand(this));
         CancelCommand = new RelayCommand(_ => cancelHandler(this));
diff --git a/Source/NETworkManager/ViewModels/OKMessageViewModel.cs b/Source/NETworkManager/ViewModels/OKMessageViewModel.cs
index 3547003cb6..9044ec66c7 100644
--- a/Source/NETworkManager/ViewModels/OKMessageViewModel.cs
+++ b/Source/NETworkManager/ViewModels/OKMessageViewModel.cs
@@ -6,7 +6,7 @@ namespace NETworkManager.ViewModels;
 
 public class OKMessageViewModel : ViewModelBase
 {
-    public OKMessageViewModel(Action<OKMessageViewModel> okCommand, string message, string okButtonText = null, ChildWindowIcon icon = ChildWindowIcon.Info)
+    public OKMessageViewModel(Action<OKMessageViewModel> okCommand, string message, ChildWindowIcon icon = ChildWindowIcon.Info, string okButtonText = null)
     {
         OKCommand = new RelayCommand(_ => okCommand(this));
 
diff --git a/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
index 7aae52a9ca..fa7be54e44 100644
--- a/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
@@ -234,9 +234,9 @@ public async Task EditPortProfile()
 
     private Task DeletePortProfile()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -247,7 +247,7 @@ private Task DeletePortProfile()
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
-            Strings.DeletePortProfileMessage, Strings.Delete);
+            Strings.DeletePortProfileMessage, ChildWindowIcon.Info, Strings.Delete);
 
         childWindow.Title = Strings.DeletePortProfile;
 
diff --git a/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs b/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs
index 883bb6f366..88c91b637b 100644
--- a/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs
@@ -203,9 +203,9 @@ public async Task EditOIDProfile()
 
     private Task DeleteOIDProfile()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -216,7 +216,7 @@ private Task DeleteOIDProfile()
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
-            Strings.DeleteOIDProfileMessage, Strings.Delete);
+            Strings.DeleteOIDProfileMessage, ChildWindowIcon.Info, Strings.Delete);
 
         childWindow.Title = Strings.DeleteOIDProfile;
 
diff --git a/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs b/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs
index 518bb12dcf..5252888082 100644
--- a/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs
@@ -187,9 +187,9 @@ public async Task EditServer()
 
     private Task DeleteServer()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -200,7 +200,7 @@ private Task DeleteServer()
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
-            Strings.DeleteSNTPServerMessage, Strings.Delete);
+            Strings.DeleteSNTPServerMessage, ChildWindowIcon.Info, Strings.Delete);
 
         childWindow.Title = Strings.DeleteSNTPServer;
 
diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index 78e507d865..109e292724 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -140,39 +140,12 @@ private async Task AddProfileFileAction()
             .FirstOrDefault(p => p.Name.Equals(profileName, StringComparison.OrdinalIgnoreCase));
 
         // Ask to enable encryption for the new profile file
-        if (await ShowEnableEncryptionMessage())
-            EnableEncryptionAction();
-    }
-
-    private async Task<bool> ShowEnableEncryptionMessage()
-    {
-        var result = false;
-
-        var childWindow = new OKCancelInfoMessageChildWindow();
-
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
-        {
-            childWindow.IsOpen = false;
-            ConfigurationManager.Current.IsChildWindowOpen = false;
-
-            result = true;
-        }, _ =>
-        {
-            childWindow.IsOpen = false;
-            ConfigurationManager.Current.IsChildWindowOpen = false;
-        },
-            Strings.EnableEncryptionForProfileFileMessage
-        );
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow, 
+            Strings.EnableEncryptionQuestion, 
+            Strings.EnableEncryptionForProfileFileMessage);
 
-        childWindow.Title = Strings.EnableEncryptionQuestion;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
-
-        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
-
-        return result;
+        if (result)
+            EnableEncryptionAction();
     }
 
     public ICommand EditProfileFileCommand => new RelayCommand(async _ => await EditProfileFileAction().ConfigureAwait(false));
@@ -222,9 +195,9 @@ private bool DeleteProfileFile_CanExecute(object obj)
 
     private Task DeleteProfileFileAction()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -235,7 +208,7 @@ private Task DeleteProfileFileAction()
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
-           string.Format(Strings.DeleteProfileFileXMessage, SelectedProfileFile.Name), Strings.Delete);
+           string.Format(Strings.DeleteProfileFileXMessage, SelectedProfileFile.Name), ChildWindowIcon.Info, Strings.Delete);
 
         childWindow.Title = Strings.DeleteProfileFile;
 
@@ -289,9 +262,9 @@ private async Task<bool> ShowEncryptionDisclaimerAsync()
     {
         var result = false;
 
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
         {
             childWindow.IsOpen = false;
             ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -391,11 +364,11 @@ await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
                     ChildWindowIcon.Error).ConfigureAwait(false);
             }
 
-            }, _ =>
-        {
-            childWindow.IsOpen = false;
-            ConfigurationManager.Current.IsChildWindowOpen = false;
-        });
+        }, _ =>
+    {
+        childWindow.IsOpen = false;
+        ConfigurationManager.Current.IsChildWindowOpen = false;
+    });
 
         childWindow.Title = Strings.MasterPassword;
 
@@ -404,7 +377,7 @@ await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
         ConfigurationManager.Current.IsChildWindowOpen = true;
 
         return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
-    }    
+    }
 
     #endregion
 }
\ No newline at end of file
diff --git a/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs b/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs
index aef50dee2f..7dc0a130bd 100644
--- a/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs
@@ -68,9 +68,9 @@ private void ResetSettingsAction()
 
     private Task ResetSettings()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(_ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
         {
             childWindow.IsOpen = false;
             ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -86,9 +86,9 @@ private Task ResetSettings()
             ConfigurationManager.Current.IsChildWindowOpen = false;
         },
             Strings.SettingsAreResetAndApplicationWillBeRestartedMessage,
+            ChildWindowIcon.Question,
             Strings.Reset,
-            Strings.Cancel,
-            ChildWindowIcon.Question
+            Strings.Cancel            
         );
 
         childWindow.Title = Strings.ResetSettingsQuestion;
diff --git a/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs b/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs
index 397412bae5..7578499fb9 100644
--- a/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs
@@ -108,9 +108,9 @@ private void DeleteBrowsingDataAction()
 
     private Task DeleteBrowsingData()
     {
-        var childWindow = new OKCancelInfoMessageChildWindow();
+        var childWindow = new OKCancelMessageChildWindow();
 
-        var childWindowViewModel = new OKCancelInfoMessageViewModel(async _ =>
+        var childWindowViewModel = new OKCancelMessageViewModel(async _ =>
             {
                 childWindow.IsOpen = false;
                 ConfigurationManager.Current.IsChildWindowOpen = false;
@@ -132,6 +132,7 @@ private Task DeleteBrowsingData()
                 ConfigurationManager.Current.IsChildWindowOpen = false;
             },
             Strings.DeleteBrowsingDataMessage,
+            ChildWindowIcon.Info, 
             Strings.Delete
         );
 
diff --git a/Source/NETworkManager/Views/OKCancelInfoMessageChildWindow.xaml b/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml
similarity index 97%
rename from Source/NETworkManager/Views/OKCancelInfoMessageChildWindow.xaml
rename to Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml
index 480e739d72..370e84189b 100644
--- a/Source/NETworkManager/Views/OKCancelInfoMessageChildWindow.xaml
+++ b/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml
@@ -1,4 +1,4 @@
-<simpleChildWindow:ChildWindow x:Class="NETworkManager.Views.OKCancelInfoMessageChildWindow"
+<simpleChildWindow:ChildWindow x:Class="NETworkManager.Views.OKCancelMessageChildWindow"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
@@ -10,7 +10,7 @@
              xmlns:converters="clr-namespace:NETworkManager.Converters;assembly=NETworkManager.Converters"
              Loaded="ChildWindow_OnLoaded"
              Style="{StaticResource DefaultChildWindow}"
-             mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:OKCancelInfoMessageViewModel}">
+             mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:OKCancelMessageViewModel}">
     <simpleChildWindow:ChildWindow.Resources>
         <converters:ChildWindowIconToRectangleStyleConverter x:Key="ChildWindowIconToRectangleStyleConverter" />
     </simpleChildWindow:ChildWindow.Resources>
diff --git a/Source/NETworkManager/Views/OKCancelInfoMessageChildWindow.xaml.cs b/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml.cs
similarity index 79%
rename from Source/NETworkManager/Views/OKCancelInfoMessageChildWindow.xaml.cs
rename to Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml.cs
index 5853058f48..6ce7afa025 100644
--- a/Source/NETworkManager/Views/OKCancelInfoMessageChildWindow.xaml.cs
+++ b/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml.cs
@@ -3,9 +3,9 @@
 
 namespace NETworkManager.Views;
 
-public partial class OKCancelInfoMessageChildWindow
+public partial class OKCancelMessageChildWindow
 {
-    public OKCancelInfoMessageChildWindow()
+    public OKCancelMessageChildWindow()
     {
         InitializeComponent();
     }

From faf0c2bbb29745397bd36d656f0c66edd138255c Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Sun, 9 Nov 2025 01:05:27 +0100
Subject: [PATCH 03/10] Chore: Refactor dialogs

---
 .../ViewModels/SettingsProfilesViewModel.cs   | 120 +++++++++---------
 ...CredentialsChangePasswordChildWindow.xaml} |  31 +++--
 ...dentialsChangePasswordChildWindow.xaml.cs} |   4 +-
 3 files changed, 76 insertions(+), 79 deletions(-)
 rename Source/NETworkManager/Views/{CredentialsChangePasswordDialog.xaml => CredentialsChangePasswordChildWindow.xaml} (80%)
 rename Source/NETworkManager/Views/{CredentialsChangePasswordDialog.xaml.cs => CredentialsChangePasswordChildWindow.xaml.cs} (78%)

diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index 109e292724..19e3ad1107 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -133,15 +133,16 @@ private async Task AddProfileFileAction()
 
         await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
 
+        // Re-select the profile file
         if (string.IsNullOrEmpty(profileName))
             return;
 
         SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>()
             .FirstOrDefault(p => p.Name.Equals(profileName, StringComparison.OrdinalIgnoreCase));
 
-        // Ask to enable encryption for the new profile file
-        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow, 
-            Strings.EnableEncryptionQuestion, 
+        // Ask the user if they want to enable encryption for the new profile file
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.EnableEncryptionQuestion,
             Strings.EnableEncryptionForProfileFileMessage);
 
         if (result)
@@ -178,6 +179,7 @@ private async Task EditProfileFileAction()
 
         await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
 
+        // Re-select the profile file
         if (string.IsNullOrEmpty(profileName))
             return;
 
@@ -193,7 +195,7 @@ private bool DeleteProfileFile_CanExecute(object obj)
         return ProfileFiles.Cast<ProfileFileInfo>().Count() > 1;
     }
 
-    private Task DeleteProfileFileAction()
+    private async Task DeleteProfileFileAction()
     {
         var childWindow = new OKCancelMessageChildWindow();
 
@@ -216,14 +218,20 @@ private Task DeleteProfileFileAction()
 
         ConfigurationManager.Current.IsChildWindowOpen = true;
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+
+        // Select the first profile file
+        SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>().FirstOrDefault();
     }
 
     public ICommand EnableEncryptionCommand => new RelayCommand(_ => EnableEncryptionAction());
 
     private async void EnableEncryptionAction()
     {
-        if (!await ShowEncryptionDisclaimerAsync())
+        // Show encryption disclaimer
+        if (!await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+             Strings.Disclaimer,
+             Strings.ProfileEncryptionDisclaimer))
             return;
 
         var customDialog = new CustomDialog
@@ -257,50 +265,19 @@ await _dialogCoordinator.ShowMessageAsync(this, Strings.EncryptionError,
 
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
+   
+    public ICommand ChangeMasterPasswordCommand => new RelayCommand(async _ => await ChangeMasterPasswordAction().ConfigureAwait(false));
 
-    private async Task<bool> ShowEncryptionDisclaimerAsync()
+    private async Task ChangeMasterPasswordAction()
     {
-        var result = false;
-
-        var childWindow = new OKCancelMessageChildWindow();
+        var profileName = SelectedProfileFile.Name;
 
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-        {
-            childWindow.IsOpen = false;
-            ConfigurationManager.Current.IsChildWindowOpen = false;
+        var childWindow = new CredentialsChangePasswordChildWindow();
 
-            result = true;
-        }, _ =>
+        var childWindowViewModel = new CredentialsChangePasswordViewModel(async instance =>
         {
             childWindow.IsOpen = false;
             ConfigurationManager.Current.IsChildWindowOpen = false;
-        },
-            Strings.ProfileEncryptionDisclaimer
-        );
-
-        childWindow.Title = Strings.Disclaimer;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
-
-        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
-
-        return result;
-    }
-
-    public ICommand ChangeMasterPasswordCommand => new RelayCommand(_ => ChangeMasterPasswordAction());
-
-    private async void ChangeMasterPasswordAction()
-    {
-        var customDialog = new CustomDialog
-        {
-            Title = Strings.ChangeMasterPassword
-        };
-
-        var credentialsPasswordViewModel = new CredentialsChangePasswordViewModel(async instance =>
-        {
-            await _dialogCoordinator.HideMetroDialogAsync(this, customDialog);
 
             try
             {
@@ -308,36 +285,46 @@ private async void ChangeMasterPasswordAction()
             }
             catch (CryptographicException)
             {
-                var settings = AppearanceManager.MetroDialog;
-                settings.AffirmativeButtonText = Strings.OK;
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.WrongPassword,
-                    Strings.WrongPasswordDecryptionFailedMessage, MessageDialogStyle.Affirmative,
-                    settings);
+                await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
+                    Strings.WrongPassword,
+                    Strings.WrongPasswordDecryptionFailedMessage,
+                    ChildWindowIcon.Error).ConfigureAwait(false);
             }
             catch (Exception ex)
             {
-                var settings = AppearanceManager.MetroDialog;
-                settings.AffirmativeButtonText = Strings.OK;
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.DecryptionError,
+                await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
+                    Strings.DecryptionError,
                     $"{Strings.DecryptionErrorMessage}\n\n{ex.Message}",
-                    MessageDialogStyle.Affirmative, settings);
+                    ChildWindowIcon.Error).ConfigureAwait(false);
             }
-        }, async _ => { await _dialogCoordinator.HideMetroDialogAsync(this, customDialog); });
-
-        customDialog.Content = new CredentialsChangePasswordDialog
+        }, _ =>
         {
-            DataContext = credentialsPasswordViewModel
-        };
+            childWindow.IsOpen = false;
+            ConfigurationManager.Current.IsChildWindowOpen = false;
+        });
 
-        await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
+        childWindow.Title = Strings.ChangeMasterPassword;
+
+        childWindow.DataContext = childWindowViewModel;
+
+        ConfigurationManager.Current.IsChildWindowOpen = true;
+
+        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+
+        // Re-select the profile file
+        if (string.IsNullOrEmpty(profileName))
+            return;
+
+        SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>()
+            .FirstOrDefault(p => p.Name.Equals(profileName, StringComparison.OrdinalIgnoreCase));
     }
 
     public ICommand DisableEncryptionCommand => new RelayCommand(async _ => await DisableEncryptionAction().ConfigureAwait(false));
 
-    private Task DisableEncryptionAction()
+    private async Task DisableEncryptionAction()
     {
+        var profileName = SelectedProfileFile.Name;
+
         var childWindow = new CredentialsPasswordChildWindow();
 
         var childWindowViewModel = new CredentialsPasswordViewModel(async instance =>
@@ -376,8 +363,15 @@ await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
 
         ConfigurationManager.Current.IsChildWindowOpen = true;
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+
+        // Re-select the profile file
+        if (string.IsNullOrEmpty(profileName))
+            return;
+                
+        SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>()
+            .FirstOrDefault(p => p.Name.Equals(profileName, StringComparison.OrdinalIgnoreCase));
     }
 
     #endregion
-}
\ No newline at end of file
+}
diff --git a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml b/Source/NETworkManager/Views/CredentialsChangePasswordChildWindow.xaml
similarity index 80%
rename from Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml
rename to Source/NETworkManager/Views/CredentialsChangePasswordChildWindow.xaml
index 57ab8188d1..f1d9287417 100644
--- a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml
+++ b/Source/NETworkManager/Views/CredentialsChangePasswordChildWindow.xaml
@@ -1,4 +1,4 @@
-<UserControl x:Class="NETworkManager.Views.CredentialsChangePasswordDialog"
+<simpleChildWindow:ChildWindow x:Class="NETworkManager.Views.CredentialsChangePasswordChildWindow"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
@@ -7,9 +7,12 @@
              xmlns:interactivity="http://schemas.microsoft.com/xaml/behaviors"
              xmlns:viewModels="clr-namespace:NETworkManager.ViewModels"
              xmlns:localization="clr-namespace:NETworkManager.Localization.Resources;assembly=NETworkManager.Localization"
-             mc:Ignorable="d" Loaded="ChildWindow_OnLoaded"
-             d:DataContext="{d:DesignInstance viewModels:CredentialsChangePasswordViewModel}">
-    <Grid Margin="0,20">
+             xmlns:simpleChildWindow="clr-namespace:MahApps.Metro.SimpleChildWindow;assembly=MahApps.Metro.SimpleChildWindow"             
+             CloseButtonCommand="{Binding Path=CancelCommand}"
+             Style="{StaticResource DefaultChildWindow}"
+             Loaded="ChildWindow_OnLoaded"                            
+             mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:CredentialsChangePasswordViewModel}">    
+    <Grid Margin="10">
         <Grid.RowDefinitions>
             <RowDefinition Height="*" />
             <RowDefinition Height="Auto" />
@@ -35,31 +38,31 @@
             </Grid.RowDefinitions>
             <TextBlock Grid.Column="0" Grid.Row="0" Text="{x:Static localization:Strings.CurrentPassword}" />
             <PasswordBox x:Name="PasswordBoxPassword" Grid.Column="2" Grid.Row="0"
-                         Style="{StaticResource DefaultPasswordBox}">
+                     Style="{StaticResource DefaultPasswordBox}">
                 <interactivity:Interaction.Behaviors>
                     <wpfHelpers:PasswordBoxBindingBehavior
-                        Password="{Binding Password, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
+                    Password="{Binding Password, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
                 </interactivity:Interaction.Behaviors>
             </PasswordBox>
             <TextBlock Grid.Column="0" Grid.Row="2" Text="{x:Static localization:Strings.NewPassword}" />
             <PasswordBox x:Name="PasswordBoxNewPassword" Grid.Column="2" Grid.Row="2"
-                         Style="{StaticResource DefaultPasswordBox}">
+                     Style="{StaticResource DefaultPasswordBox}">
                 <interactivity:Interaction.Behaviors>
                     <wpfHelpers:PasswordBoxBindingBehavior
-                        Password="{Binding NewPassword, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
+                    Password="{Binding NewPassword, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
                 </interactivity:Interaction.Behaviors>
             </PasswordBox>
             <TextBlock Grid.Column="0" Grid.Row="4" Text="{x:Static localization:Strings.Repeat}" />
             <PasswordBox x:Name="PasswordBoxNewPasswordRepeat" Grid.Column="2" Grid.Row="4"
-                         Style="{StaticResource DefaultPasswordBox}">
+                     Style="{StaticResource DefaultPasswordBox}">
                 <interactivity:Interaction.Behaviors>
                     <wpfHelpers:PasswordBoxBindingBehavior
-                        Password="{Binding NewPasswordRepeat, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
+                    Password="{Binding NewPasswordRepeat, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
                 </interactivity:Interaction.Behaviors>
             </PasswordBox>
         </Grid>
         <TextBlock Grid.Row="1" Foreground="{DynamicResource MahApps.Brushes.Accent}"
-                   Text="{x:Static localization:Strings.PasswordsDoNotMatch}" Margin="0,10,0,0">
+               Text="{x:Static localization:Strings.PasswordsDoNotMatch}" Margin="0,10,0,0">
             <TextBlock.Style>
                 <Style TargetType="{x:Type TextBlock}" BasedOn="{StaticResource DefaultTextBlock}">
                     <Setter Property="Visibility" Value="Collapsed" />
@@ -79,7 +82,7 @@
         </TextBlock>
         <StackPanel Grid.Row="3" Orientation="Horizontal" HorizontalAlignment="Right">
             <Button Content="{x:Static localization:Strings.OK}" Command="{Binding OKCommand}" IsDefault="True"
-                    Margin="0,0,10,0">
+                Margin="0,0,10,0">
                 <Button.Style>
                     <Style TargetType="{x:Type Button}" BasedOn="{StaticResource HighlightedButton}">
                         <Setter Property="IsEnabled" Value="False" />
@@ -98,7 +101,7 @@
                 </Button.Style>
             </Button>
             <Button Content="{x:Static localization:Strings.Cancel}" Command="{Binding CancelCommand}" IsCancel="True"
-                    Style="{StaticResource DefaultButton}" />
+                Style="{StaticResource DefaultButton}" />
         </StackPanel>
     </Grid>
-</UserControl>
\ No newline at end of file
+</simpleChildWindow:ChildWindow>
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs b/Source/NETworkManager/Views/CredentialsChangePasswordChildWindow.xaml.cs
similarity index 78%
rename from Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs
rename to Source/NETworkManager/Views/CredentialsChangePasswordChildWindow.xaml.cs
index f9bb4c192c..942027ec56 100644
--- a/Source/NETworkManager/Views/CredentialsChangePasswordDialog.xaml.cs
+++ b/Source/NETworkManager/Views/CredentialsChangePasswordChildWindow.xaml.cs
@@ -4,9 +4,9 @@
 
 namespace NETworkManager.Views;
 
-public partial class CredentialsChangePasswordDialog
+public partial class CredentialsChangePasswordChildWindow
 {
-    public CredentialsChangePasswordDialog()
+    public CredentialsChangePasswordChildWindow()
     {
         InitializeComponent();
     }

From 9257e1d126cfae95c6d292f506a361df848d0a60 Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Sun, 9 Nov 2025 01:12:03 +0100
Subject: [PATCH 04/10] Feature: Migrate dialog

---
 .../ViewModels/SettingsProfilesViewModel.cs   | 52 +++++++++++--------
 ...=> CredentialsSetPasswordChildWindow.xaml} | 27 +++++-----
 .../CredentialsSetPasswordChildWindow.xaml.cs | 21 ++++++++
 .../CredentialsSetPasswordDialog.xaml.cs      | 16 ------
 4 files changed, 67 insertions(+), 49 deletions(-)
 rename Source/NETworkManager/Views/{CredentialsSetPasswordDialog.xaml => CredentialsSetPasswordChildWindow.xaml} (81%)
 create mode 100644 Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml.cs
 delete mode 100644 Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml.cs

diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index 19e3ad1107..5e972a4a87 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -224,9 +224,9 @@ private async Task DeleteProfileFileAction()
         SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>().FirstOrDefault();
     }
 
-    public ICommand EnableEncryptionCommand => new RelayCommand(_ => EnableEncryptionAction());
+    public ICommand EnableEncryptionCommand => new RelayCommand(async _ => await EnableEncryptionAction().ConfigureAwait(false));
 
-    private async void EnableEncryptionAction()
+    private async Task EnableEncryptionAction()
     {
         // Show encryption disclaimer
         if (!await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
@@ -234,38 +234,48 @@ private async void EnableEncryptionAction()
              Strings.ProfileEncryptionDisclaimer))
             return;
 
-        var customDialog = new CustomDialog
-        {
-            Title = Strings.SetMasterPassword
-        };
 
-        var credentialsSetPasswordViewModel = new CredentialsSetPasswordViewModel(async instance =>
-        {
-            await _dialogCoordinator.HideMetroDialogAsync(this, customDialog);
+        var profileFile = SelectedProfileFile.Name;
 
+        var childWindow = new CredentialsSetPasswordChildWindow();
+
+        var childWindowViewModel = new CredentialsSetPasswordViewModel(async instance =>
+        {
+            childWindow.IsOpen = false;
+            ConfigurationManager.Current.IsChildWindowOpen = false;
             try
             {
                 ProfileManager.EnableEncryption(SelectedProfileFile, instance.Password);
             }
             catch (Exception ex)
             {
-                var metroDialogSettings = AppearanceManager.MetroDialog;
-                metroDialogSettings.AffirmativeButtonText = Strings.OK;
-
-                await _dialogCoordinator.ShowMessageAsync(this, Strings.EncryptionError,
+                await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
+                    Strings.EncryptionError,
                     $"{Strings.EncryptionErrorMessage}\n\n{ex.Message}",
-                    MessageDialogStyle.Affirmative, metroDialogSettings);
+                    ChildWindowIcon.Error).ConfigureAwait(false);
             }
-        }, async _ => { await _dialogCoordinator.HideMetroDialogAsync(this, customDialog); });
-
-        customDialog.Content = new CredentialsSetPasswordDialog
+        }, _ =>
         {
-            DataContext = credentialsSetPasswordViewModel
-        };
+            childWindow.IsOpen = false;
+            ConfigurationManager.Current.IsChildWindowOpen = false;
+        });
+
+        childWindow.Title = Strings.SetMasterPassword;
+
+        childWindow.DataContext = childWindowViewModel;
 
-        await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
+        ConfigurationManager.Current.IsChildWindowOpen = true;
+
+        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+
+        // Re-select the profile file
+        if (string.IsNullOrEmpty(profileFile))
+            return;
+
+        SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>()
+            .FirstOrDefault(p => p.Name.Equals(profileFile, StringComparison.OrdinalIgnoreCase));
     }
-   
+
     public ICommand ChangeMasterPasswordCommand => new RelayCommand(async _ => await ChangeMasterPasswordAction().ConfigureAwait(false));
 
     private async Task ChangeMasterPasswordAction()
diff --git a/Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml b/Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml
similarity index 81%
rename from Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml
rename to Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml
index f1657782d3..962e6d0496 100644
--- a/Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml
+++ b/Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml
@@ -1,4 +1,4 @@
-<UserControl x:Class="NETworkManager.Views.CredentialsSetPasswordDialog"
+<simpleChildWindow:ChildWindow x:Class="NETworkManager.Views.CredentialsSetPasswordChildWindow"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
@@ -7,9 +7,12 @@
              xmlns:interactivity="http://schemas.microsoft.com/xaml/behaviors"
              xmlns:viewModels="clr-namespace:NETworkManager.ViewModels"
              xmlns:localization="clr-namespace:NETworkManager.Localization.Resources;assembly=NETworkManager.Localization"
-             mc:Ignorable="d" Loaded="UserControl_Loaded"
-             d:DataContext="{d:DesignInstance viewModels:CredentialsSetPasswordViewModel}">
-    <Grid Margin="0,20">
+             xmlns:simpleChildWindow="clr-namespace:MahApps.Metro.SimpleChildWindow;assembly=MahApps.Metro.SimpleChildWindow"             
+             CloseButtonCommand="{Binding Path=CancelCommand}"
+             Style="{StaticResource DefaultChildWindow}"
+             Loaded="ChildWindow_OnLoaded"                            
+             mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:CredentialsSetPasswordViewModel}">    
+    <Grid Margin="10">
         <Grid.RowDefinitions>
             <RowDefinition Height="*" />
             <RowDefinition Height="Auto" />
@@ -33,23 +36,23 @@
             </Grid.RowDefinitions>
             <TextBlock Grid.Column="0" Grid.Row="0" Text="{x:Static localization:Strings.Password}" />
             <PasswordBox x:Name="PasswordBoxPassword" Grid.Column="2" Grid.Row="0"
-                         Style="{StaticResource DefaultPasswordBox}">
+                   Style="{StaticResource DefaultPasswordBox}">
                 <interactivity:Interaction.Behaviors>
                     <wpfHelpers:PasswordBoxBindingBehavior
-                        Password="{Binding Password, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
+                  Password="{Binding Password, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
                 </interactivity:Interaction.Behaviors>
             </PasswordBox>
             <TextBlock Grid.Column="0" Grid.Row="2" Text="{x:Static localization:Strings.Repeat}" />
             <PasswordBox x:Name="PasswordBoxPasswordRepeat" Grid.Column="2" Grid.Row="2"
-                         Style="{StaticResource DefaultPasswordBox}">
+                   Style="{StaticResource DefaultPasswordBox}">
                 <interactivity:Interaction.Behaviors>
                     <wpfHelpers:PasswordBoxBindingBehavior
-                        Password="{Binding PasswordRepeat, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
+                  Password="{Binding PasswordRepeat, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />
                 </interactivity:Interaction.Behaviors>
             </PasswordBox>
         </Grid>
         <TextBlock Grid.Row="1" Foreground="{DynamicResource MahApps.Brushes.Accent}"
-                   Text="{x:Static localization:Strings.PasswordsDoNotMatch}" Margin="0,10,0,0">
+             Text="{x:Static localization:Strings.PasswordsDoNotMatch}" Margin="0,10,0,0">
             <TextBlock.Style>
                 <Style TargetType="{x:Type TextBlock}" BasedOn="{StaticResource DefaultTextBlock}">
                     <Setter Property="Visibility" Value="Collapsed" />
@@ -69,7 +72,7 @@
         </TextBlock>
         <StackPanel Grid.Row="3" Orientation="Horizontal" HorizontalAlignment="Right">
             <Button Content="{x:Static localization:Strings.OK}" Command="{Binding OKCommand}" IsDefault="True"
-                    Margin="0,0,10,0">
+              Margin="0,0,10,0">
                 <Button.Style>
                     <Style TargetType="{x:Type Button}" BasedOn="{StaticResource HighlightedButton}">
                         <Setter Property="IsEnabled" Value="False" />
@@ -88,7 +91,7 @@
                 </Button.Style>
             </Button>
             <Button Content="{x:Static localization:Strings.Cancel}" Command="{Binding CancelCommand}" IsCancel="True"
-                    Style="{StaticResource DefaultButton}" />
+              Style="{StaticResource DefaultButton}" />
         </StackPanel>
     </Grid>
-</UserControl>
\ No newline at end of file
+</simpleChildWindow:ChildWindow>
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml.cs b/Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml.cs
new file mode 100644
index 0000000000..072bd53883
--- /dev/null
+++ b/Source/NETworkManager/Views/CredentialsSetPasswordChildWindow.xaml.cs
@@ -0,0 +1,21 @@
+using System;
+using System.Windows;
+using System.Windows.Threading;
+
+namespace NETworkManager.Views;
+
+public partial class CredentialsSetPasswordChildWindow
+{
+    public CredentialsSetPasswordChildWindow()
+    {
+        InitializeComponent();
+    }
+
+    private void ChildWindow_OnLoaded(object sender, RoutedEventArgs e)
+    {
+        Dispatcher.BeginInvoke(DispatcherPriority.ContextIdle, new Action(delegate
+        {
+            PasswordBoxPassword.Focus();
+        }));
+    }
+}
\ No newline at end of file
diff --git a/Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml.cs b/Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml.cs
deleted file mode 100644
index 298e0943d1..0000000000
--- a/Source/NETworkManager/Views/CredentialsSetPasswordDialog.xaml.cs
+++ /dev/null
@@ -1,16 +0,0 @@
-using System.Windows;
-
-namespace NETworkManager.Views;
-
-public partial class CredentialsSetPasswordDialog
-{
-    public CredentialsSetPasswordDialog()
-    {
-        InitializeComponent();
-    }
-
-    private void UserControl_Loaded(object sender, RoutedEventArgs e)
-    {
-        PasswordBoxPassword.Focus();
-    }
-}
\ No newline at end of file

From 34bfbfcdfe818b9028cf61f4d44f715c47db8eb6 Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Sun, 9 Nov 2025 03:26:47 +0100
Subject: [PATCH 05/10] Chore: Migrate to new dialog helper

---
 .../SecureStringHelper.cs                     |  1 +
 .../PortScannerSettingsViewModel.cs           | 34 ++++++++-----------
 2 files changed, 15 insertions(+), 20 deletions(-)

diff --git a/Source/NETworkManager.Utilities/SecureStringHelper.cs b/Source/NETworkManager.Utilities/SecureStringHelper.cs
index a9e50f7489..e3370badb0 100644
--- a/Source/NETworkManager.Utilities/SecureStringHelper.cs
+++ b/Source/NETworkManager.Utilities/SecureStringHelper.cs
@@ -9,6 +9,7 @@ public static class SecureStringHelper
     public static string ConvertToString(SecureString secureString)
     {
         var valuePtr = IntPtr.Zero;
+
         try
         {
             valuePtr = Marshal.SecureStringToGlobalAllocUnicode(secureString);
diff --git a/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
index fa7be54e44..fadc89fa04 100644
--- a/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
@@ -6,6 +6,7 @@
 using NETworkManager.Utilities;
 using NETworkManager.Views;
 using System.ComponentModel;
+using System.Linq;
 using System.Threading.Tasks;
 using System.Windows;
 using System.Windows.Data;
@@ -142,6 +143,8 @@ public PortScannerSettingsViewModel(IDialogCoordinator instance)
         PortProfiles.SortDescriptions.Add(
             new SortDescription(nameof(PortProfileInfo.Name), ListSortDirection.Ascending));
 
+        SelectedPortProfile = PortProfiles.Cast<PortProfileInfo>().FirstOrDefault();
+
         LoadSettings();
 
         _isLoading = false;
@@ -232,30 +235,21 @@ public async Task EditPortProfile()
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    private Task DeletePortProfile()
+    private async Task DeletePortProfile()
     {
-        var childWindow = new OKCancelMessageChildWindow();
-
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                SettingsManager.Current.PortScanner_PortProfiles.Remove(SelectedPortProfile);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-            Strings.DeletePortProfileMessage, ChildWindowIcon.Info, Strings.Delete);
-
-        childWindow.Title = Strings.DeletePortProfile;
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+               Strings.DeletePortProfile,
+               Strings.DeletePortProfileMessage,
+               ChildWindowIcon.Info,
+               Strings.Delete);
 
-        childWindow.DataContext = childWindowViewModel;
+        if (!result)
+            return;
 
-        ConfigurationManager.Current.IsChildWindowOpen = true;
+        SettingsManager.Current.PortScanner_PortProfiles.Remove(SelectedPortProfile);
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        // Select first item after deletion
+        SelectedPortProfile = PortProfiles.Cast<PortProfileInfo>().FirstOrDefault();
     }
 
     #endregion

From 090dd9b8fabe38917cbf502b4e1dced3962c8473 Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Mon, 10 Nov 2025 23:05:40 +0100
Subject: [PATCH 06/10] Feature: Redesign dialogs

---
 Source/NETworkManager/ProfileDialogManager.cs | 75 ++++++-------------
 .../AWSSessionManagerSettingsViewModel.cs     | 32 +++-----
 .../ViewModels/DNSLookupSettingsViewModel.cs  | 33 +++-----
 .../ViewModels/HostsFileEditorViewModel.cs    | 41 ++++------
 .../ViewModels/IPScannerSettingsViewModel.cs  | 33 +++-----
 .../ViewModels/SNMPSettingsViewModel.cs       | 30 +++-----
 .../ViewModels/SNTPLookupSettingsViewModel.cs | 30 +++-----
 .../ViewModels/SettingsProfilesViewModel.cs   | 28 ++-----
 .../ViewModels/SettingsSettingsViewModel.cs   | 36 +++------
 .../ViewModels/WebConsoleSettingsViewModel.cs | 45 ++++-------
 .../Views/OKCancelMessageChildWindow.xaml     |  3 +-
 .../Views/OKMessageChildWindow.xaml           |  1 +
 12 files changed, 117 insertions(+), 270 deletions(-)

diff --git a/Source/NETworkManager/ProfileDialogManager.cs b/Source/NETworkManager/ProfileDialogManager.cs
index 5c17ff6c95..d1e48ab2d0 100644
--- a/Source/NETworkManager/ProfileDialogManager.cs
+++ b/Source/NETworkManager/ProfileDialogManager.cs
@@ -561,40 +561,24 @@ public static Task ShowCopyAsProfileDialog(Window parentWindow, IProfileManagerM
         return parentWindow.ShowChildWindowAsync(childWindow);
     }
 
-    public static Task ShowDeleteProfileDialog(Window parentWindow, IProfileManagerMinimal viewModel,
+    public static async Task ShowDeleteProfileDialog(Window parentWindow, IProfileManagerMinimal viewModel,
         IList<ProfileInfo> profiles)
     {
-        var childWindow = new OKCancelMessageChildWindow();
 
-        OKCancelMessageViewModel childWindowViewModel = new(_ =>
-            {
-                childWindow.IsOpen = false;
-                Settings.ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                viewModel.OnProfileManagerDialogClose();
-
-                ProfileManager.RemoveProfiles(profiles);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                Settings.ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                viewModel.OnProfileManagerDialogClose();
-            },
-                profiles.Count == 1 ? Strings.DeleteProfileMessage : Strings.DeleteProfilesMessage,
-                Utilities.ChildWindowIcon.Info,
-                Strings.Delete
-            );
-
-        childWindow.Title = profiles.Count == 1 ? Strings.DeleteProfile : Strings.DeleteProfiles;
+        viewModel.OnProfileManagerDialogOpen();
 
-        childWindow.DataContext = childWindowViewModel;
+        var result = await DialogHelper.ShowOKCancelMessageAsync(parentWindow,
+            profiles.Count == 1 ? Strings.DeleteProfile : Strings.DeleteProfiles,
+            profiles.Count == 1 ? Strings.DeleteProfileMessage : Strings.DeleteProfilesMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        viewModel.OnProfileManagerDialogOpen();
+        viewModel.OnProfileManagerDialogClose();
 
-        Settings.ConfigurationManager.Current.IsChildWindowOpen = true;
+        if (!result)
+            return;
 
-        return parentWindow.ShowChildWindowAsync(childWindow);
+        ProfileManager.RemoveProfiles(profiles);
     }
 
     #endregion
@@ -664,40 +648,23 @@ public static Task ShowEditGroupDialog(Window parentWindow, IProfileManagerMinim
         return parentWindow.ShowChildWindowAsync(childWindow);
     }
 
-    public static Task ShowDeleteGroupDialog(Window parentWindow, IProfileManagerMinimal viewModel,
+    public static async Task ShowDeleteGroupDialog(Window parentWindow, IProfileManagerMinimal viewModel,
         GroupInfo group)
     {
-        var childWindow = new OKCancelMessageChildWindow();
-
-        OKCancelMessageViewModel childWindowViewModel = new(_ =>
-        {
-            childWindow.IsOpen = false;
-            Settings.ConfigurationManager.Current.IsChildWindowOpen = false;
-
-            viewModel.OnProfileManagerDialogClose();
-
-            ProfileManager.RemoveGroup(group);
-        }, _ =>
-        {
-            childWindow.IsOpen = false;
-            Settings.ConfigurationManager.Current.IsChildWindowOpen = false;
+        viewModel.OnProfileManagerDialogOpen();
 
-            viewModel.OnProfileManagerDialogClose();
-        },
+        var result = await DialogHelper.ShowOKCancelMessageAsync(parentWindow,
+            Strings.DeleteGroup,
             Strings.DeleteGroupMessage,
-            ChildWindowIcon.Info, 
-            Strings.Delete
-        );
-
-        childWindow.Title = Strings.DeleteGroup;
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        childWindow.DataContext = childWindowViewModel;
+        if (!result)
+            return;
 
-        viewModel.OnProfileManagerDialogOpen();
+        viewModel.OnProfileManagerDialogClose();
 
-        Settings.ConfigurationManager.Current.IsChildWindowOpen = true;
-
-        return parentWindow.ShowChildWindowAsync(childWindow);
+        ProfileManager.RemoveGroup(group);
     }
 
     #endregion
diff --git a/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs
index 87b942eb2c..fe0809c4c1 100644
--- a/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/AWSSessionManagerSettingsViewModel.cs
@@ -275,33 +275,19 @@ public async Task EditAWSProfile()
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    private Task DeleteAWSProfile()
+    private async Task DeleteAWSProfile()
     {
-        var childWindow = new OKCancelMessageChildWindow();
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteAWSProfile,
+            Strings.DeleteAWSProfileMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
 
-                SettingsManager.Current.AWSSessionManager_AWSProfiles.Remove(SelectedAWSProfile);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-                Strings.DeleteAWSProfileMessage,
-                ChildWindowIcon.Info,
-                Strings.Delete
-            );
+        if (!result)
+            return;
 
-        childWindow.Title = Strings.DeleteAWSProfile;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
-
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        SettingsManager.Current.AWSSessionManager_AWSProfiles.Remove(SelectedAWSProfile);
     }
 
     private async Task Configure()
diff --git a/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs b/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs
index 5dc6f03428..1bcfe63d1d 100644
--- a/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/DNSLookupSettingsViewModel.cs
@@ -357,33 +357,18 @@ public async Task EditDNSServer()
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    private Task DeleteDNSServer()
+    private async Task DeleteDNSServer()
     {
-        var childWindow = new OKCancelMessageChildWindow();
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteDNSServer,
+            Strings.DeleteDNSServerMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                SettingsManager.Current.DNSLookup_DNSServers.Remove(SelectedDNSServer);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-                Strings.DeleteDNSServerMessage,
-                ChildWindowIcon.Info,
-                Strings.Delete
-            );
-
-        childWindow.Title = Strings.DeleteDNSServer;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
+        if (!result)
+            return;
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        SettingsManager.Current.DNSLookup_DNSServers.Remove(SelectedDNSServer);
     }
 
     #endregion
diff --git a/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs b/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
index 85881e987e..5db830cf20 100644
--- a/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
+++ b/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
@@ -391,39 +391,28 @@ private async Task DeleteEntryAction()
     {
         IsModifying = true;
 
-        var childWindow = new OKCancelMessageChildWindow();
-
-        var childWindowViewModel = new OKCancelMessageViewModel(async _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                var result = await HostsFileEditor.DeleteEntryAsync(SelectedResult);
-
-                if (result != HostsFileEntryModifyResult.Success)
-                    await ShowErrorMessageAsync(result);
-
-                IsModifying = false;
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                IsModifying = false;
-            },
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteEntry,
             string.Format(Strings.DeleteHostsFileEntryMessage, SelectedResult.IPAddress, SelectedResult.Hostname,
                 string.IsNullOrEmpty(SelectedResult.Comment) ? "" : $"# {SelectedResult.Comment}"),
             ChildWindowIcon.Info,
-            Strings.Delete
-        );
+                        Strings.Delete
+            );
 
-        childWindow.Title = Strings.DeleteEntry;
 
-        childWindow.DataContext = childWindowViewModel;
+        if (!result)
+        {
+            IsModifying = false;
+            return;
+        }
 
-        ConfigurationManager.Current.IsChildWindowOpen = true;
 
-        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        var modifyResult = await HostsFileEditor.DeleteEntryAsync(SelectedResult);
+
+        if (modifyResult != HostsFileEntryModifyResult.Success)
+            await ShowErrorMessageAsync(modifyResult);
+
+        IsModifying = false;
     }
 
     private bool ModifyEntry_CanExecute(object obj)
diff --git a/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs
index 35cb90c59c..9c8095bd90 100644
--- a/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/IPScannerSettingsViewModel.cs
@@ -385,33 +385,18 @@ public async void EditCustomCommand()
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    private Task DeleteCustomCommand()
+    private async Task DeleteCustomCommand()
     {
-        var childWindow = new OKCancelMessageChildWindow();
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteCustomCommand,
+            Strings.DeleteCustomCommandMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
+        if (!result)
+            return;
 
-                SettingsManager.Current.IPScanner_CustomCommands.Remove(SelectedCustomCommand);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-                Strings.DeleteCustomCommandMessage,
-                ChildWindowIcon.Info,
-                Strings.Delete
-            );
-
-        childWindow.Title = Strings.DeleteCustomCommand;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
-
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        SettingsManager.Current.IPScanner_CustomCommands.Remove(SelectedCustomCommand);
     }
 
     #endregion
diff --git a/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs b/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs
index 88c91b637b..70b7c4d1f1 100644
--- a/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SNMPSettingsViewModel.cs
@@ -201,30 +201,18 @@ public async Task EditOIDProfile()
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    private Task DeleteOIDProfile()
+    private async Task DeleteOIDProfile()
     {
-        var childWindow = new OKCancelMessageChildWindow();
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteOIDProfile,
+            Strings.DeleteOIDProfileMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
+        if (!result)
+            return;
 
-                SettingsManager.Current.SNMP_OidProfiles.Remove(SelectedOIDProfile);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-            Strings.DeleteOIDProfileMessage, ChildWindowIcon.Info, Strings.Delete);
-
-        childWindow.Title = Strings.DeleteOIDProfile;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
-
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        SettingsManager.Current.SNMP_OidProfiles.Remove(SelectedOIDProfile);
     }
 
     #endregion
diff --git a/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs b/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs
index 5252888082..09ac160119 100644
--- a/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SNTPLookupSettingsViewModel.cs
@@ -185,30 +185,18 @@ public async Task EditServer()
         await _dialogCoordinator.ShowMetroDialogAsync(this, customDialog);
     }
 
-    private Task DeleteServer()
+    private async Task DeleteServer()
     {
-        var childWindow = new OKCancelMessageChildWindow();
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteSNTPServer,
+            Strings.DeleteSNTPServerMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                SettingsManager.Current.SNTPLookup_SNTPServers.Remove(SelectedSNTPServer);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-            Strings.DeleteSNTPServerMessage, ChildWindowIcon.Info, Strings.Delete);
-
-        childWindow.Title = Strings.DeleteSNTPServer;
-
-        childWindow.DataContext = childWindowViewModel;
-
-        ConfigurationManager.Current.IsChildWindowOpen = true;
+        if (!result)
+            return;
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        SettingsManager.Current.SNTPLookup_SNTPServers.Remove(SelectedSNTPServer);
     }
 
     #endregion
diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index 5e972a4a87..f612977ee4 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -197,28 +197,16 @@ private bool DeleteProfileFile_CanExecute(object obj)
 
     private async Task DeleteProfileFileAction()
     {
-        var childWindow = new OKCancelMessageChildWindow();
-
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                ProfileManager.DeleteProfileFile(SelectedProfileFile);
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-           string.Format(Strings.DeleteProfileFileXMessage, SelectedProfileFile.Name), ChildWindowIcon.Info, Strings.Delete);
-
-        childWindow.Title = Strings.DeleteProfileFile;
-
-        childWindow.DataContext = childWindowViewModel;
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteProfileFile,
+            string.Format(Strings.DeleteProfileFileXMessage, SelectedProfileFile.Name),
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-        ConfigurationManager.Current.IsChildWindowOpen = true;
+        if (!result)
+            return;
 
-        await (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        ProfileManager.DeleteProfileFile(SelectedProfileFile);
 
         // Select the first profile file
         SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>().FirstOrDefault();
diff --git a/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs b/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs
index 7dc0a130bd..d004a0d3bb 100644
--- a/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs
@@ -66,38 +66,22 @@ private void ResetSettingsAction()
 
     #region Methods
 
-    private Task ResetSettings()
+    private async Task ResetSettings()
     {
-        var childWindow = new OKCancelMessageChildWindow();
-
-        var childWindowViewModel = new OKCancelMessageViewModel(_ =>
-        {
-            childWindow.IsOpen = false;
-            ConfigurationManager.Current.IsChildWindowOpen = false;
-
-            // Init default settings
-            SettingsManager.Initialize();
-
-            // Restart the application
-            (Application.Current.MainWindow as MainWindow)?.RestartApplication();
-        }, _ =>
-        {
-            childWindow.IsOpen = false;
-            ConfigurationManager.Current.IsChildWindowOpen = false;
-        },
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.ResetSettingsQuestion,
             Strings.SettingsAreResetAndApplicationWillBeRestartedMessage,
             ChildWindowIcon.Question,
-            Strings.Reset,
-            Strings.Cancel            
-        );
-
-        childWindow.Title = Strings.ResetSettingsQuestion;
+            Strings.Reset);
 
-        childWindow.DataContext = childWindowViewModel;
+        if (!result)
+            return;
 
-        ConfigurationManager.Current.IsChildWindowOpen = true;
+        // Init default settings
+        SettingsManager.Initialize();
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        // Restart the application
+        (Application.Current.MainWindow as MainWindow)?.RestartApplication();
     }
     #endregion
 }
diff --git a/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs b/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs
index 7578499fb9..fcde9483e5 100644
--- a/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/WebConsoleSettingsViewModel.cs
@@ -106,43 +106,28 @@ private void DeleteBrowsingDataAction()
 
     #region Methods
 
-    private Task DeleteBrowsingData()
+    private async Task DeleteBrowsingData()
     {
-        var childWindow = new OKCancelMessageChildWindow();
-
-        var childWindowViewModel = new OKCancelMessageViewModel(async _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-
-                // Create a temporary WebView2 instance to clear browsing data
-                var webView2Environment =
-                    await CoreWebView2Environment.CreateAsync(null, GlobalStaticConfiguration.WebConsole_Cache);
-
-                var windowHwnd = new WindowInteropHelper(Application.Current.MainWindow).Handle;
-
-                var webView2Controller = await webView2Environment.CreateCoreWebView2ControllerAsync(windowHwnd);
+        var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
+            Strings.DeleteBrowsingData,
+            Strings.DeleteBrowsingDataMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
-                await webView2Controller.CoreWebView2.Profile.ClearBrowsingDataAsync();
+        if (!result)
+            return;
 
-                webView2Controller.Close();
-            }, _ =>
-            {
-                childWindow.IsOpen = false;
-                ConfigurationManager.Current.IsChildWindowOpen = false;
-            },
-            Strings.DeleteBrowsingDataMessage,
-            ChildWindowIcon.Info, 
-            Strings.Delete
-        );
+        // Create a temporary WebView2 instance to clear browsing data
+        var webView2Environment =
+            await CoreWebView2Environment.CreateAsync(null, GlobalStaticConfiguration.WebConsole_Cache);
 
-        childWindow.Title = Strings.DeleteBrowsingData;
+        var windowHwnd = new WindowInteropHelper(Application.Current.MainWindow).Handle;
 
-        childWindow.DataContext = childWindowViewModel;
+        var webView2Controller = await webView2Environment.CreateCoreWebView2ControllerAsync(windowHwnd);
 
-        ConfigurationManager.Current.IsChildWindowOpen = true;
+        await webView2Controller.CoreWebView2.Profile.ClearBrowsingDataAsync();
 
-        return (Application.Current.MainWindow as MainWindow).ShowChildWindowAsync(childWindow);
+        webView2Controller.Close();
     }
 
     #endregion
diff --git a/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml b/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml
index 370e84189b..c205459443 100644
--- a/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml
+++ b/Source/NETworkManager/Views/OKCancelMessageChildWindow.xaml
@@ -9,7 +9,8 @@
              xmlns:iconPacks="http://metro.mahapps.com/winfx/xaml/iconpacks"
              xmlns:converters="clr-namespace:NETworkManager.Converters;assembly=NETworkManager.Converters"
              Loaded="ChildWindow_OnLoaded"
-             Style="{StaticResource DefaultChildWindow}"
+             CloseButtonCommand="{Binding Path=CancelCommand}"
+             Style="{StaticResource DefaultChildWindow}"                               
              mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:OKCancelMessageViewModel}">
     <simpleChildWindow:ChildWindow.Resources>
         <converters:ChildWindowIconToRectangleStyleConverter x:Key="ChildWindowIconToRectangleStyleConverter" />
diff --git a/Source/NETworkManager/Views/OKMessageChildWindow.xaml b/Source/NETworkManager/Views/OKMessageChildWindow.xaml
index d743b51e67..97df645cb8 100644
--- a/Source/NETworkManager/Views/OKMessageChildWindow.xaml
+++ b/Source/NETworkManager/Views/OKMessageChildWindow.xaml
@@ -9,6 +9,7 @@
              xmlns:iconPacks="http://metro.mahapps.com/winfx/xaml/iconpacks"
              xmlns:converters="clr-namespace:NETworkManager.Converters;assembly=NETworkManager.Converters"             
              Loaded="ChildWindow_OnLoaded"
+             CloseButtonCommand="{Binding Path=OKCommand}"
              Style="{StaticResource DefaultChildWindow}"
              mc:Ignorable="d" d:DataContext="{d:DesignInstance viewModels:OKMessageViewModel}">
     <simpleChildWindow:ChildWindow.Resources>

From 13dfd95522464ed79101e1d4b1871ffe3227864b Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Mon, 10 Nov 2025 23:09:34 +0100
Subject: [PATCH 07/10] Update next-release.md

---
 Website/docs/changelog/next-release.md | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Website/docs/changelog/next-release.md b/Website/docs/changelog/next-release.md
index 61e52ecca2..d06032b5cf 100644
--- a/Website/docs/changelog/next-release.md
+++ b/Website/docs/changelog/next-release.md
@@ -29,6 +29,7 @@ Release date: **xx.xx.2025**
 
 - Profile file creation flow improved — when adding a new profile you are now prompted to enable profile-file encryption to protect stored credentials and settings. [#3227](https://github.com/BornToBeRoot/NETworkManager/pull/3227)
 - Profile file dialog migrated to a child window to improve usability. [#3227](https://github.com/BornToBeRoot/NETworkManager/pull/3227)
+- Credential dialogs migrated to child windows to improve usability. [#3231](https://github.com/BornToBeRoot/NETworkManager/pull/3231)
 
 **Remote Desktop**
 
@@ -41,6 +42,8 @@ Release date: **xx.xx.2025**
 ## Dependencies, Refactoring & Documentation
 
 - Documentation updated
-- Code cleanup & refactoring
+- Code cleanup & refactoring 
+    - Introduced a new DialogHelper utility to centralize creation of `OK` and `OK/Cancel` dialogs. This reduces duplication and enforces a consistent layout. [#3231](https://github.com/BornToBeRoot/NETworkManager/pull/3231)
+- Migrated existing credential and profile dialogs to use DialogHelper, improving usability, accessibility and maintainability across the app. [#3231](https://github.com/BornToBeRoot/NETworkManager/pull/3231)
 - Language files updated via [#transifex](https://github.com/BornToBeRoot/NETworkManager/pulls?q=author%3Aapp%2Ftransifex-integration)
 - Dependencies updated via [#dependabot](https://github.com/BornToBeRoot/NETworkManager/pulls?q=author%3Aapp%2Fdependabot)

From 7382c7dc922804094167814b3d0ca65886e941a3 Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Mon, 10 Nov 2025 23:16:07 +0100
Subject: [PATCH 08/10] Update
 Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---
 .../ViewModels/SettingsProfilesViewModel.cs               | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index f612977ee4..1a9a3e05eb 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -350,10 +350,10 @@ await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
             }
 
         }, _ =>
-    {
-        childWindow.IsOpen = false;
-        ConfigurationManager.Current.IsChildWindowOpen = false;
-    });
+            {
+                childWindow.IsOpen = false;
+                ConfigurationManager.Current.IsChildWindowOpen = false;
+            });
 
         childWindow.Title = Strings.MasterPassword;
 

From 843f9edb15e890779ff5faf47bf1548d465bd434 Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Tue, 11 Nov 2025 10:09:32 +0100
Subject: [PATCH 09/10] Remove duplicate call to OnProfileManagerDialogClose

---
 Source/NETworkManager/ProfileDialogManager.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Source/NETworkManager/ProfileDialogManager.cs b/Source/NETworkManager/ProfileDialogManager.cs
index d1e48ab2d0..a9d5a7e80c 100644
--- a/Source/NETworkManager/ProfileDialogManager.cs
+++ b/Source/NETworkManager/ProfileDialogManager.cs
@@ -659,13 +659,13 @@ public static async Task ShowDeleteGroupDialog(Window parentWindow, IProfileMana
             ChildWindowIcon.Info,
             Strings.Delete);
 
+        viewModel.OnProfileManagerDialogClose();
+        
         if (!result)
             return;
 
-        viewModel.OnProfileManagerDialogClose();
-
         ProfileManager.RemoveGroup(group);
     }
 
     #endregion
-}
\ No newline at end of file
+}

From 1a3a96fd6ba135e1f2f21d11601633ec85a9ef8d Mon Sep 17 00:00:00 2001
From: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Date: Tue, 11 Nov 2025 11:42:11 +0100
Subject: [PATCH 10/10] Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---
 Source/NETworkManager/ProfileDialogManager.cs             | 2 +-
 .../NETworkManager/ViewModels/HostsFileEditorViewModel.cs | 2 +-
 .../ViewModels/PortScannerSettingsViewModel.cs            | 8 ++++----
 .../ViewModels/SettingsProfilesViewModel.cs               | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/Source/NETworkManager/ProfileDialogManager.cs b/Source/NETworkManager/ProfileDialogManager.cs
index a9d5a7e80c..d36342fdff 100644
--- a/Source/NETworkManager/ProfileDialogManager.cs
+++ b/Source/NETworkManager/ProfileDialogManager.cs
@@ -660,7 +660,7 @@ public static async Task ShowDeleteGroupDialog(Window parentWindow, IProfileMana
             Strings.Delete);
 
         viewModel.OnProfileManagerDialogClose();
-        
+
         if (!result)
             return;
 
diff --git a/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs b/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
index 5db830cf20..041d5e720b 100644
--- a/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
+++ b/Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs
@@ -396,7 +396,7 @@ private async Task DeleteEntryAction()
             string.Format(Strings.DeleteHostsFileEntryMessage, SelectedResult.IPAddress, SelectedResult.Hostname,
                 string.IsNullOrEmpty(SelectedResult.Comment) ? "" : $"# {SelectedResult.Comment}"),
             ChildWindowIcon.Info,
-                        Strings.Delete
+            Strings.Delete
             );
 
 
diff --git a/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs b/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
index fadc89fa04..d3b10a7ada 100644
--- a/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
+++ b/Source/NETworkManager/ViewModels/PortScannerSettingsViewModel.cs
@@ -238,10 +238,10 @@ public async Task EditPortProfile()
     private async Task DeletePortProfile()
     {
         var result = await DialogHelper.ShowOKCancelMessageAsync(Application.Current.MainWindow,
-               Strings.DeletePortProfile,
-               Strings.DeletePortProfileMessage,
-               ChildWindowIcon.Info,
-               Strings.Delete);
+            Strings.DeletePortProfile,
+            Strings.DeletePortProfileMessage,
+            ChildWindowIcon.Info,
+            Strings.Delete);
 
         if (!result)
             return;
diff --git a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
index 1a9a3e05eb..f6bd5260f7 100644
--- a/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
+++ b/Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs
@@ -366,7 +366,7 @@ await DialogHelper.ShowOKMessageAsync(Application.Current.MainWindow,
         // Re-select the profile file
         if (string.IsNullOrEmpty(profileName))
             return;
-                
+
         SelectedProfileFile = ProfileFiles.Cast<ProfileFileInfo>()
             .FirstOrDefault(p => p.Name.Equals(profileName, StringComparison.OrdinalIgnoreCase));
     }