Refactor HostEditor and HostList UI components to enhance user interaction and input validation. Changed port and server alive settings from EntryRow to SpinRow for numeric input, added button actions for SSH command copying and connection testing, and improved error handling for input values. Removed unused count label from HostList to streamline the interface.

Author: BuddySirJava

data/ui/host_editor.blp

@@ -128,8 +128,16 @@ template $HostEditor: Gtk.Box {
             title: _("Username");
           }
 
-          Adw.EntryRow port_entry {
+          Adw.SpinRow port_entry {
             title: _("Port");
+            numeric: true;
+            adjustment: Gtk.Adjustment {
+              value: 22;
+              lower: 1;
+              upper: 65535;
+              step-increment: 1;
+              page-increment: 10;
+            };
           }
 
           Label port_error_label {
@@ -156,6 +164,9 @@ template $HostEditor: Gtk.Box {
               margin-bottom: 8;
               icon-name: "document-open-symbolic";
               tooltip-text: _("Choose Identity File");
+              styles [
+                "flat",
+              ]
             }
 
             [suffix]
@@ -164,6 +175,9 @@ template $HostEditor: Gtk.Box {
               margin-bottom: 8;
               icon-name: "dialog-password-symbolic";
               tooltip-text: _("Pick from existing SSH keys");
+              styles [
+                "flat",
+              ]
             }
           }
 
@@ -181,16 +195,28 @@ template $HostEditor: Gtk.Box {
         Adw.PreferencesGroup {
           title: _("Actions");
 
-          Adw.ActionRow copy_row {
-            title: _("Copy SSH command");
-            activatable: true;
-            subtitle: _("Copies the resolved SSH command to the clipboard");
-          }
+          Box {
+            orientation: horizontal;
+            spacing: 12;
+            margin-top: 12;
+            margin-bottom: 12;
+
+            Button copy_button {
+              label: _("Copy SSH command");
+              hexpand: true;
+              styles [
+                "pill",
+                "suggested-action",
+              ]
+            }
 
-          Adw.ActionRow test_row {
-            title: _("Test connection");
-            activatable: true;
-            subtitle: _("Runs a quick non-interactive SSH check");
+            Button test_button {
+              label: _("Test connection");
+              hexpand: true;
+              styles [
+                "pill",
+              ]
+            }
           }
         }
       };
@@ -247,16 +273,28 @@ template $HostEditor: Gtk.Box {
             }
           }
 
-          Adw.EntryRow serveralive_interval_entry {
+          Adw.SpinRow serveralive_interval_entry {
             title: _("ServerAliveInterval (s)");
-            show-apply-button: false;
-            input-purpose: number;
+            numeric: true;
+            adjustment: Gtk.Adjustment {
+              value: 0;
+              lower: 0;
+              upper: 300;
+              step-increment: 1;
+              page-increment: 10;
+            };
           }
 
-          Adw.EntryRow serveralive_count_entry {
+          Adw.SpinRow serveralive_count_entry {
             title: _("ServerAliveCountMax");
-            show-apply-button: false;
-            input-purpose: number;
+            numeric: true;
+            adjustment: Gtk.Adjustment {
+              value: 3;
+              lower: 1;
+              upper: 100;
+              step-increment: 1;
+              page-increment: 5;
+            };
           }
 
           Adw.ActionRow {

data/ui/host_list.blp

@@ -37,7 +37,6 @@ template $HostList: Box {
         label: _("SSH Studio");
         halign: center;
         hexpand: true;
-        margin-start: 24;
         styles [
           "title-4",
         ]
@@ -116,13 +115,6 @@ template $HostList: Box {
       }
     }
 
-    Label count_label {
-      label: _("0 hosts");
-      halign: end;
-      styles [
-        "dim-label",
-      ]
-    }
 
   }
 }

src/main.py

@@ -189,12 +189,6 @@ def update_ui():
                             self.main_window.host_list.load_hosts(
                                 self.parser.config.hosts
                             )
-                            try:
-                                self.main_window._update_status(
-                                    _("Configuration loaded successfully")
-                                )
-                            except Exception:
-                                pass
                     except Exception:
                         pass
                     return False