Finalize robustness pass and set minimum size to 119x35

Author: skibidiandulka

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "nettui"
-version = "0.1.8"
+version = "0.1.9"
 edition = "2024"
 description = "Unified TUI for Wi-Fi and Ethernet"
 license = "GPL-3.0-only"

Cargo.toml

@@ -25,7 +25,7 @@ This project was inspired by and builds upon ideas from Impala by pythops.
   - link up/down toggle on selected interface
   - toggle link admin state up/down (`ip link set`)
   - DHCP renew (`networkctl renew`)
-- Toast/error popups and terminal size guard
+- Toast/error popups and terminal size guard (`119x35` minimum)
 
 ## Runtime assumptions
 
@@ -111,11 +111,41 @@ cargo test
 
 ## Omarchy integration (optional)
 
-Recommended launch command for Omarchy-style app-id handling:
+Current Omarchy launcher behavior:
 
 ```bash
-omarchy-launch-or-focus-tui nettui
+omarchy-launch-wifi
 ```
 
-If Omarchy chooses to make `nettui` the default network TUI later, this command can become the single
-launcher entry for both Wi-Fi and Ethernet workflows.
+On recent Omarchy, this already prefers `nettui` when installed.
+
+To force `nettui` as default network TUI:
+
+1. Install `nettui`:
+
+```bash
+yay -S nettui-bin
+```
+
+2. Verify launcher script prefers `nettui`:
+
+```bash
+grep -n "nettui" ~/.local/share/omarchy/bin/omarchy-launch-wifi
+grep -n "nettui" ~/.local/share/omarchy/bin/omarchy-launch-ethernet
+```
+
+3. If needed, patch both launchers:
+
+```bash
+sed -i 's/omarchy-launch-or-focus-tui impala/omarchy-launch-or-focus-tui nettui/g' ~/.local/share/omarchy/bin/omarchy-launch-wifi
+sed -i 's/omarchy-launch-or-focus-tui ethtui/omarchy-launch-or-focus-tui nettui/g' ~/.local/share/omarchy/bin/omarchy-launch-ethernet
+```
+
+4. Set dedicated floating window size for `nettui` (Hyprland):
+
+```bash
+grep -q "match:class org.omarchy.nettui" ~/.config/hypr/apps/system.conf || echo "windowrule = size 1190 735, match:class org.omarchy.nettui" >> ~/.config/hypr/apps/system.conf
+hyprctl reload
+```
+
+5. Click network module in Waybar to verify `nettui` opens.

README.md

@@ -844,11 +844,11 @@ impl App {
     }
 
     fn restore_ethernet_selection(&mut self, selected_iface: Option<String>) {
-        if let Some(name) = selected_iface {
-            if let Some(idx) = self.ethernet.ifaces.iter().position(|i| i.name == name) {
-                self.ethernet_state.select(Some(idx));
-                return;
-            }
+        if let Some(name) = selected_iface
+            && let Some(idx) = self.ethernet.ifaces.iter().position(|i| i.name == name)
+        {
+            self.ethernet_state.select(Some(idx));
+            return;
         }
         select_first_if_any(&mut self.ethernet_state, self.ethernet.ifaces.len());
     }
@@ -889,9 +889,7 @@ pub fn determine_start_tab(
         StartupTabPolicy::PreferActive => {
             if ethernet.has_active() {
                 ActiveTab::Ethernet
-            } else if wifi.is_active() {
-                ActiveTab::Wifi
-            } else if wifi.has_adapter() {
+            } else if wifi.is_active() || wifi.has_adapter() {
                 ActiveTab::Wifi
             } else {
                 ActiveTab::Ethernet

src/app.rs

@@ -296,6 +296,12 @@ impl IwdBackend {
     }
 }
 
+impl Default for IwdBackend {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 #[derive(Debug, Clone)]
 struct KnownMeta {
     security: String,

src/backend/iwd.rs

@@ -45,21 +45,21 @@ impl NetworkdBackend {
                     .output()
                     .await;
 
-                if let Ok(pkexec_out) = pkexec_out {
-                    if pkexec_out.status.success() {
-                        return Ok(CommandResult {
-                            program: "networkctl".to_string(),
-                            args: vec!["renew".to_string(), iface.to_string()],
-                            used_sudo: true,
-                            status: pkexec_out.status.code().unwrap_or(0),
-                            stdout: String::from_utf8_lossy(&pkexec_out.stdout)
-                                .trim()
-                                .to_string(),
-                            stderr: String::from_utf8_lossy(&pkexec_out.stderr)
-                                .trim()
-                                .to_string(),
-                        });
-                    }
+                if let Ok(pkexec_out) = pkexec_out
+                    && pkexec_out.status.success()
+                {
+                    return Ok(CommandResult {
+                        program: "networkctl".to_string(),
+                        args: vec!["renew".to_string(), iface.to_string()],
+                        used_sudo: true,
+                        status: pkexec_out.status.code().unwrap_or(0),
+                        stdout: String::from_utf8_lossy(&pkexec_out.stdout)
+                            .trim()
+                            .to_string(),
+                        stderr: String::from_utf8_lossy(&pkexec_out.stderr)
+                            .trim()
+                            .to_string(),
+                    });
                 }
 
                 let sudo_out = Command::new("sudo")
@@ -142,27 +142,27 @@ impl NetworkdBackend {
                     .output()
                     .await;
 
-                if let Ok(pkexec_out) = pkexec_out {
-                    if pkexec_out.status.success() {
-                        return Ok(CommandResult {
-                            program: "ip".to_string(),
-                            args: vec![
-                                "link".to_string(),
-                                "set".to_string(),
-                                "dev".to_string(),
-                                iface.to_string(),
-                                state_arg.to_string(),
-                            ],
-                            used_sudo: true,
-                            status: pkexec_out.status.code().unwrap_or(0),
-                            stdout: String::from_utf8_lossy(&pkexec_out.stdout)
-                                .trim()
-                                .to_string(),
-                            stderr: String::from_utf8_lossy(&pkexec_out.stderr)
-                                .trim()
-                                .to_string(),
-                        });
-                    }
+                if let Ok(pkexec_out) = pkexec_out
+                    && pkexec_out.status.success()
+                {
+                    return Ok(CommandResult {
+                        program: "ip".to_string(),
+                        args: vec![
+                            "link".to_string(),
+                            "set".to_string(),
+                            "dev".to_string(),
+                            iface.to_string(),
+                            state_arg.to_string(),
+                        ],
+                        used_sudo: true,
+                        status: pkexec_out.status.code().unwrap_or(0),
+                        stdout: String::from_utf8_lossy(&pkexec_out.stdout)
+                            .trim()
+                            .to_string(),
+                        stderr: String::from_utf8_lossy(&pkexec_out.stderr)
+                            .trim()
+                            .to_string(),
+                    });
                 }
 
                 let sudo_out = Command::new("sudo")
@@ -226,6 +226,12 @@ impl NetworkdBackend {
     }
 }
 
+impl Default for NetworkdBackend {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl EthernetBackend for NetworkdBackend {
     fn list_ifaces(&self) -> Result<Vec<EthernetIface>> {
         list_ethernet_ifaces()

src/backend/networkd.rs

@@ -9,8 +9,8 @@ use ratatui::{
 };
 
 pub fn render(app: &mut App, frame: &mut Frame) {
-    const MIN_W: u16 = 126;
-    const MIN_H: u16 = 27;
+    const MIN_W: u16 = 119;
+    const MIN_H: u16 = 35;
 
     let area = frame.area();
     if area.width < MIN_W || area.height < MIN_H {