Feature: #3109 - Add F2/Delete hotkeys in HostsFileEditor & Refactor

[BornToBeRoot]

Changes proposed in this pull request

• Add F2/Delete hotkeys in HostsFileEditor
• Refactor some code

Related issue(s)

• Fixes HostsFileEditor - Add F2/double click to edit entry #3109

Copilot generated summary

Provide a Copilot generated summary of the changes in this pull request.

Copilot summary

This pull request introduces several improvements to the NETworkManager project, focusing on enhancing user interface interactions, improving command execution patterns, and ensuring better accessibility by setting focus on specific UI elements when child windows are loaded. Below is a summary of the most important changes grouped by theme:

Enhanced User Interface Interactions:

• Source/NETworkManager/ViewModels/HostsFileEditorViewModel.cs: Replaced the dialog coordinator with a child window (OKMessageChildWindow) for error messages, improving consistency in UI design.
• Source/NETworkManager/Views/HostsFileEditorView.xaml: Added mouse double-click event and keyboard shortcuts (F2 for edit, Delete for delete) to the MultiSelectDataGrid for better user interaction. [1] [2]

Improved Command Execution Patterns:

• Replaced direct method calls with CanExecute checks and Execute invocations for commands across multiple files to ensure proper command validation before execution:
  • Source/NETworkManager/Views/AWSSessionManagerSettingsView.xaml.cs: Updated AWS profile editing logic.
  • Source/NETworkManager/Views/DNSLookupSettingsView.xaml.cs: Updated DNS server editing logic.
  • Source/NETworkManager/Views/HostsFileEditorView.xaml.cs: Added command execution for editing entries.
  • Similar updates in PortScannerSettingsView.xaml.cs, SNMPSettingsView.xaml.cs, SNTPLookupSettingsView.xaml.cs, and SettingsProfilesView.xaml.cs. [1] [2] [3] [4]

Accessibility Improvements in Child Windows:

• Added focus logic to specific UI elements when child windows are loaded to improve accessibility and user experience:
  • Source/NETworkManager/Views/OKMessageChildWindow.xaml and OKMessageChildWindow.xaml.cs: Focuses the OK button when loaded. [1] [2]
  • Similar focus improvements in UpgradeChildWindow, WelcomeChildWindow, and OKCancelInfoMessageChildWindow. [1] [2] [3] [4] [5]

Code Cleanup:

• Removed unnecessary comments and redundant using directives across several files to improve code readability and maintainability:
  • Source/NETworkManager/Views/DNSLookupSettingsView.xaml.cs and HostsFileEditorView.xaml.cs. [1] [2]

These changes collectively enhance the user experience, improve code quality, and ensure better adherence to UI and accessibility standards.

To-Do

• Update documentation to reflect this changes

Contributing

By submitting this pull request, I confirm the following:

• I have read and understood the contributing guidelines and the code of conduct.
• I have have added my name, username or email to the contributors list or don't want to.
• The code or resource is compatible with the GNU General Public License v3.0.