Tweaks

Author: Axenide

modules/bar.py

@@ -171,7 +171,7 @@ def __init__(self, **kwargs):
 
         self.hidden = False
 
-        self.show_all()
+        # self.show_all()
         self.systray._update_visibility()
 
     def on_button_enter(self, widget, event):

modules/buttons.py

@@ -24,7 +24,7 @@ def __init__(self):
         self._animation_timeout_id = None
         self._animation_step = 0
         self._animation_direction = 1
-        
+
         self.network_icon = Label(
             name="network-icon",
             markup=None,
@@ -51,7 +51,7 @@ def __init__(self):
             h_align="start",
             v_align="center",
             spacing=10,
-            
+
             children=[self.network_icon, self.network_text],
         )
         self.network_status_button = Button(
@@ -83,16 +83,20 @@ def __init__(self):
             children=[self.network_status_button, self.network_menu_button],
         )
 
-        self.widgets = [self, self.network_icon, self.network_label, 
-                       self.network_ssid, self.network_status_button, 
+        self.widgets = [self, self.network_icon, self.network_label,
+                       self.network_ssid, self.network_status_button,
                        self.network_menu_button, self.network_menu_label]
 
         # Connect to wifi device signals when ready
         self.network_client.connect('device-ready', self._on_wifi_ready)
-        
-        # Check initial state if wifi device is already available
-        if self.network_client.wifi_device:
-            self.update_state()
+
+        # Check initial state using idle_add to defer until GTK loop is running
+        GLib.idle_add(self._initial_update)
+
+
+    def _initial_update(self):
+        self.update_state()
+        return False  # Run only once
 
     def _on_wifi_ready(self, *args):
         if self.network_client.wifi_device:
@@ -103,23 +107,23 @@ def _on_wifi_ready(self, *args):
     def _animate_searching(self):
         """Animate wifi icon when searching for networks"""
         wifi_icons = [icons.wifi_0, icons.wifi_1, icons.wifi_2, icons.wifi_3, icons.wifi_2, icons.wifi_1]
-        
+
         # Si el widget no existe o el WiFi está desactivado, detener la animación
         wifi = self.network_client.wifi_device
         if not self.network_icon or not wifi or not wifi.enabled:
             self._stop_animation()
             return False
-            
+
         # Si estamos conectados, detener la animación
         if wifi.state == "activated" and wifi.ssid != "Disconnected":
             self._stop_animation()
             return False
-            
+
         GLib.idle_add(self.network_icon.set_markup, wifi_icons[self._animation_step])
-        
+
         # Reiniciar al principio cuando llegamos al final
         self._animation_step = (self._animation_step + 1) % len(wifi_icons)
-            
+
         return True  # Mantener la animación activa
 
     def _start_animation(self):
@@ -140,7 +144,7 @@ def update_state(self, *args):
         wifi = self.network_client.wifi_device
         ethernet = self.network_client.ethernet_device
 
-        # Primero actualizamos el estado enabled/disabled
+        # Update enabled/disabled state
         if wifi and not wifi.enabled:
             self._stop_animation()
             self.network_icon.set_markup(icons.wifi_off)
@@ -149,16 +153,16 @@ def update_state(self, *args):
             self.network_ssid.set_label("Disabled")
             return
 
-        # Removemos la clase disabled si llegamos aquí
+        # Remove disabled class if we got here
         for widget in self.widgets:
             widget.remove_style_class("disabled")
 
-        # Actualizar el texto y la animación según el estado
+        # Update text and animation based on state
         if wifi and wifi.enabled:
             if wifi.state == "activated" and wifi.ssid != "Disconnected":
                 self._stop_animation()
                 self.network_ssid.set_label(wifi.ssid)
-                # Actualizar icono según la intensidad de la señal
+                # Update icon based on signal strength
                 if wifi.strength > 0:
                     strength = wifi.strength
                     if strength < 25:
@@ -173,8 +177,14 @@ def update_state(self, *args):
                 self.network_ssid.set_label("Enabled")
                 self._start_animation()
 
-        # Manejar el caso de conexión por cable
-        if self.network_client.primary_device == "wired":
+        # Handle primary device check safely
+        try:
+            primary_device = self.network_client.primary_device
+        except AttributeError:
+            primary_device = "wireless"  # Default to wireless if error occurs
+
+        # Handle wired connection case
+        if primary_device == "wired":
             self._stop_animation()
             if ethernet and ethernet.internet == "activated":
                 self.network_icon.set_markup(icons.world)

modules/weather.py

@@ -52,6 +52,8 @@ def _fetch_weather_thread(self):
                 GLib.idle_add(self.label.set_label, weather_data.replace(" ", ""))
             else:
                 GLib.idle_add(self.label.set_markup, f"{icons.cloud_off} Unavailable")
+                self.set_visible(False)
         except Exception as e:
-            print(f"Error al obtener clima: {e}")
+            print(f"Error fetching weather: {e}")
             GLib.idle_add(self.label.set_markup, f"{icons.cloud_off} Error")
+            self.set_visible(False)