From 86c6135def476ec87210654641c0b89b5848515c Mon Sep 17 00:00:00 2001
From: LemmyCook <studio@quadbyte.net>
Date: Sat, 6 Sep 2025 16:11:16 -0400
Subject: [PATCH] Network/Wi-Fi: improvements

- Always check for ethernet status every 30s. Should not affect battery
life.
- Less aggressive scan intervals to give more times for slow adapters.
---
 Modules/Bar/Widgets/WiFi.qml |  2 +-
 Services/NetworkService.qml  | 30 ++++++++++++++++--------------
 2 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/Modules/Bar/Widgets/WiFi.qml b/Modules/Bar/Widgets/WiFi.qml
index 77f8664..fe8ff75 100644
--- a/Modules/Bar/Widgets/WiFi.qml
+++ b/Modules/Bar/Widgets/WiFi.qml
@@ -40,6 +40,6 @@ NIconButton {
       return "signal_wifi_bad"
     }
   }
-  tooltipText: "Network / Wi-Fi."
+  tooltipText: "Manage Wi-Fi."
   onClicked: PanelService.getPanel("wifiPanel")?.toggle(screen, this)
 }
diff --git a/Services/NetworkService.qml b/Services/NetworkService.qml
index c4b5820..ee22010 100644
--- a/Services/NetworkService.qml
+++ b/Services/NetworkService.qml
@@ -54,7 +54,7 @@ Singleton {
   Component.onCompleted: {
     Logger.log("Network", "Service initialized")
     syncWifiState()
-    refresh()
+    scan()
   }
 
   // Save cache with debounce
@@ -75,6 +75,16 @@ Singleton {
     onTriggered: scan()
   }
 
+  // Ethernet check timer
+  // Always running every 30s
+  Timer {
+    id: ethernetCheckTimer
+    interval: 30000
+    running: true
+    repeat: true
+    onTriggered: ethernetStateProcess.running = true
+  }
+
   // Core functions
   function syncWifiState() {
     wifiStateProcess.running = true
@@ -87,14 +97,6 @@ Singleton {
     wifiToggleProcess.running = true
   }
 
-  function refresh() {
-    ethernetStateProcess.running = true
-
-    if (Settings.data.network.wifiEnabled) {
-      scan()
-    }
-  }
-
   function scan() {
     if (scanning)
       return
@@ -206,7 +208,7 @@ Singleton {
   // Processes
   Process {
     id: ethernetStateProcess
-    running: false
+    running: true
     command: ["nmcli", "-t", "-f", "DEVICE,TYPE,STATE", "device"]
 
     stdout: StdioCollector {
@@ -408,7 +410,7 @@ Singleton {
         Logger.log("Network", `Connected to network: "${connectProcess.ssid}"`)
 
         // Still do a scan to get accurate signal and security info
-        delayedScanTimer.interval = 1000
+        delayedScanTimer.interval = 5000
         delayedScanTimer.restart()
       }
     }
@@ -522,8 +524,8 @@ Singleton {
 
         root.forgettingNetwork = ""
 
-        // Quick scan to verify the profile is gone
-        delayedScanTimer.interval = 500
+        // Scan to verify the profile is gone
+        delayedScanTimer.interval = 5000
         delayedScanTimer.restart()
       }
     }
@@ -535,7 +537,7 @@ Singleton {
           Logger.warn("Network", "Forget error: " + text)
         }
         // Still Trigger a scan even on error
-        delayedScanTimer.interval = 500
+        delayedScanTimer.interval = 1000
         delayedScanTimer.restart()
       }
     }