Merge branch 'main' into lenemter/release-8.2.4

Author: danirabbit

data/gala.metainfo.xml.in

@@ -27,6 +27,21 @@
   <update_contact>contact_at_elementary.io</update_contact>
 
   <releases>
+    <release version="8.2.4" date="2025-06-12" urgency="medium">
+      <description>
+        <p>Improvements:</p>
+          <ul>
+          <li>Updated translations</li>
+        </ul>
+      </description>
+      <issues>
+        <issue url="https://github.com/elementary/gala/issues/1264">Applications menu opened but hidden in full screen mode</issue>
+        <issue url="https://github.com/elementary/gala/issues/2438">Gala randomly segfaults while closing Libreoffice windows</issue>
+        <issue url="https://github.com/elementary/gala/issues/2450">Apps not being brought to front</issue>
+        <issue url="https://github.com/elementary/dock/issues/310">Some apps are not showing up in the dock</issue>
+      </issues>
+    </release>
+
     <release version="8.2.3" date="2025-05-20" urgency="medium">
       <description>
         <p>Improvements:</p>

lib/AppSystem.vala

@@ -14,12 +14,64 @@ public class Gala.AppSystem : GLib.Object {
     private GLib.HashTable<unowned string, Gala.App> id_to_app;
     private GLib.HashTable<string, string> startup_wm_class_to_id;
     private Gala.AppCache app_cache;
+    private string[] all_desktop_files = {};
+    private GLib.FileMonitor[]? directory_monitors;
 
     construct {
         id_to_app = new GLib.HashTable<unowned string, Gala.App> (str_hash, str_equal);
         startup_wm_class_to_id = new GLib.HashTable<string, string> (str_hash, str_equal);
         running_apps = new GLib.HashTable<Gala.App, unowned Gala.App> (null, null);
         app_cache = new AppCache ();
+
+        update_desktop_files ();
+    }
+
+    private void update_desktop_files () {
+        var data_dirs = Environment.get_system_data_dirs ();
+        data_dirs += Environment.get_user_data_dir ();
+
+        var create_monitors = directory_monitors == null;
+        if (create_monitors) {
+            directory_monitors = {};
+        }
+
+        foreach (unowned string data_dir in data_dirs) {
+            var app_dir = Path.build_filename (data_dir, "applications");
+            if (FileUtils.test (app_dir, FileTest.EXISTS)) {
+                try {
+                    foreach (var name in enumerate_children (app_dir)) {
+                        if (!name.contains ("~") && name.has_suffix (".desktop")) {
+                            all_desktop_files += name;
+                        }
+                    }
+
+                    if (!create_monitors) {
+                        continue;
+                    }
+
+                    var monitor = File.new_for_path (app_dir).monitor (GLib.FileMonitorFlags.NONE, null);
+                    monitor.changed.connect ((file, other_file, event_type) => {
+                        if (event_type == GLib.FileMonitorEvent.CHANGES_DONE_HINT) {
+                            update_desktop_files ();
+                        }
+                    });
+                    directory_monitors += monitor;
+                } catch (Error e) {
+                    debug ("Error inside %s: %s", app_dir, e.message);
+                }
+            }
+        }
+    }
+
+    private string[] enumerate_children (string dir) throws Error {
+        string[] result = {};
+        FileInfo? file_info;
+        var enumerator = File.new_for_path (dir).enumerate_children (FileAttribute.STANDARD_NAME, 0);
+        while ((file_info = enumerator.next_file ()) != null) {
+            result += file_info.get_name ();
+        }
+
+        return result;
     }
 
     public unowned Gala.App? lookup_app (string id) {
@@ -105,6 +157,20 @@ public class Gala.AppSystem : GLib.Object {
         return lookup_heuristic_basename (desktop_file);
     }
 
+    public unowned Gala.App? guess_app_by_id (string _id) {
+        var id = _id.ascii_down ();
+        unowned Gala.App? result = null;
+
+        foreach (var name in all_desktop_files) {
+            // Try to find desktop file based on the application name
+            if (name.contains (id) && (result = lookup_app (name)) != null) {
+                return result;
+            }
+        }
+
+        return null;
+    }
+
     public void notify_app_state_changed (Gala.App app) {
         if (app.state == Gala.AppState.RUNNING) {
             running_apps.insert (app, app);

lib/Drawing/StyleManager.vala

@@ -18,11 +18,16 @@ public class Gala.Drawing.StyleManager : Object {
     }
 
     [DBus (name="io.elementary.pantheon.AccountsService")]
-    private interface AccountsService : DBusProxy {
+    private interface PantheonAccountsService : DBusProxy {
         public abstract int prefers_color_scheme { get; set; }
         public abstract int prefers_accent_color { get; set; }
     }
 
+    [DBus (name="io.elementary.SettingsDaemon.AccountsService")]
+    private interface SettingsDaemonAccountsService : DBusProxy {
+        public abstract int accent_color { get; set; }
+    }
+
     private const string FDO_ACCOUNTS_NAME = "org.freedesktop.Accounts";
     private const string FDO_ACCOUNTS_PATH = "/org/freedesktop/Accounts";
 
@@ -37,10 +42,18 @@ public class Gala.Drawing.StyleManager : Object {
     public ColorScheme prefers_color_scheme { get; private set; default = LIGHT; }
     public Gdk.RGBA theme_accent_color { get; private set; default = DEFAULT_ACCENT_COLOR; }
 
-    private AccountsService? accounts_service_proxy;
+    private PantheonAccountsService? pantheon_proxy;
+    private SettingsDaemonAccountsService? settings_daemon_proxy;
 
     construct {
-        Bus.watch_name (SYSTEM, FDO_ACCOUNTS_NAME, NONE, () => connect_to_accounts_service.begin (), () => accounts_service_proxy = null);
+        Bus.watch_name (
+            SYSTEM, FDO_ACCOUNTS_NAME, NONE,
+            () => connect_to_accounts_service.begin (),
+            () => {
+                pantheon_proxy = null;
+                settings_daemon_proxy = null;
+            }
+        );
     }
 
     private async void connect_to_accounts_service () {
@@ -49,24 +62,27 @@ public class Gala.Drawing.StyleManager : Object {
 
             var path = yield accounts.find_user_by_name (Environment.get_user_name ());
 
-            accounts_service_proxy = yield Bus.get_proxy<AccountsService> (SYSTEM, FDO_ACCOUNTS_NAME, path, GET_INVALIDATED_PROPERTIES);
+            pantheon_proxy = yield Bus.get_proxy<PantheonAccountsService> (SYSTEM, FDO_ACCOUNTS_NAME, path, GET_INVALIDATED_PROPERTIES);
+            settings_daemon_proxy = yield Bus.get_proxy<SettingsDaemonAccountsService> (SYSTEM, FDO_ACCOUNTS_NAME, path, GET_INVALIDATED_PROPERTIES);
         } catch {
             warning ("Could not connect to AccountsService. Default accent color will be used");
             return;
         }
 
-        update_color_scheme (accounts_service_proxy.prefers_color_scheme);
-        update_color (accounts_service_proxy.prefers_accent_color);
+        update_color_scheme (pantheon_proxy.prefers_color_scheme);
+        update_color (settings_daemon_proxy.accent_color);
 
-        accounts_service_proxy.g_properties_changed.connect ((changed, invalid) => {
-            var value = changed.lookup_value ("PrefersAccentColor", new VariantType ("i"));
+        pantheon_proxy.g_properties_changed.connect ((changed, invalid) => {
+            var value = changed.lookup_value ("PrefersColorScheme", new VariantType ("i"));
             if (value != null) {
-                update_color (value.get_int32 ());
+                update_color_scheme (value.get_int32 ());
             }
+        });
 
-            value = changed.lookup_value ("PrefersColorScheme", new VariantType ("i"));
+        settings_daemon_proxy.g_properties_changed.connect ((changed, invalid) => {
+            var value = changed.lookup_value ("AccentColor", new VariantType ("i"));
             if (value != null) {
-                update_color_scheme (value.get_int32 ());
+                update_color (value.get_int32 ());
             }
         });
     }

po/es.po

@@ -8,7 +8,7 @@ msgstr ""
 "Project-Id-Version: gala\n"
 "Report-Msgid-Bugs-To: https://github.com/elementary/gala/issues\n"
 "POT-Creation-Date: 2025-06-09 20:49+0000\n"
-"PO-Revision-Date: 2025-05-18 04:55+0000\n"
+"PO-Revision-Date: 2025-06-15 15:55+0000\n"
 "Last-Translator: Italo Felipe Capasso Ballesteros <[email protected]>\n"
 "Language-Team: Spanish <https://l10n.elementaryos.org/projects/desktop/gala/"
 "es/>\n"
@@ -295,14 +295,13 @@ msgstr "Captura de pantalla de %s"
 #: src/ScreenshotManager.vala:171
 #, c-format
 msgid "Open in %s"
-msgstr ""
+msgstr "Abrir en %s"
 
 #. / TRANSLATORS: %s represents a name of file manager
 #: src/ScreenshotManager.vala:178
-#, fuzzy, c-format
-#| msgid "Show in Files"
+#, c-format
 msgid "Show in %s"
-msgstr "Mostar en Archivos"
+msgstr "Mostrar en %s"
 
 #: src/ScreenshotManager.vala:189 src/ScreenshotManager.vala:548
 msgid "Screenshot taken"

po/hu.po

@@ -8,7 +8,7 @@ msgstr ""
 "Project-Id-Version: noise\n"
 "Report-Msgid-Bugs-To: https://github.com/elementary/gala/issues\n"
 "POT-Creation-Date: 2025-06-09 20:49+0000\n"
-"PO-Revision-Date: 2025-04-23 10:55+0000\n"
+"PO-Revision-Date: 2025-06-26 09:55+0000\n"
 "Last-Translator: TomiOhl <[email protected]>\n"
 "Language-Team: Hungarian <https://l10n.elementaryos.org/projects/desktop/"
 "gala/hu/>\n"
@@ -17,7 +17,7 @@ msgstr ""
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=2; plural=n != 1;\n"
-"X-Generator: Weblate 5.10.4\n"
+"X-Generator: Weblate 5.11.4\n"
 "X-Launchpad-Export-Date: 2017-02-21 05:47+0000\n"
 
 #: daemon/DBus.vala:84 daemon-gtk3/BackgroundMenu.vala:11
@@ -288,14 +288,13 @@ msgstr "Képernyőkép %s"
 #: src/ScreenshotManager.vala:171
 #, c-format
 msgid "Open in %s"
-msgstr ""
+msgstr "Megnyitás ebben: %s"
 
 #. / TRANSLATORS: %s represents a name of file manager
 #: src/ScreenshotManager.vala:178
-#, fuzzy, c-format
-#| msgid "Show in Files"
+#, c-format
 msgid "Show in %s"
-msgstr "Megjelenítés a Fájlkezelőben"
+msgstr "Megjelenítés itt: %s"
 
 #: src/ScreenshotManager.vala:189 src/ScreenshotManager.vala:548
 msgid "Screenshot taken"

src/DesktopIntegration.vala

@@ -37,10 +37,7 @@ public class Gala.DesktopIntegration : GLib.Object {
         unowned var display = wm.get_display ();
         unowned var workspace_manager = display.get_workspace_manager ();
 
-        workspace_manager.active_workspace_changed.connect (() => {
-            active_workspace_changed ();
-            windows_changed (); // windows have 'on-active-workspace' property that we need to update
-        });
+        workspace_manager.active_workspace_changed.connect (() => active_workspace_changed ());
         workspace_manager.workspaces_reordered.connect (() => windows_changed ());
         workspace_manager.workspace_added.connect (() => windows_changed ());
         workspace_manager.workspace_removed.connect ((index) => {
@@ -99,7 +96,6 @@ public class Gala.DesktopIntegration : GLib.Object {
     public Window[] get_windows () throws GLib.DBusError, GLib.IOError {
         Window[] returned_windows = {};
         var apps = Gala.AppSystem.get_default ().get_running_apps ();
-        unowned var active_workspace = wm.get_display ().get_workspace_manager ().get_active_workspace ();
         foreach (unowned var app in apps) {
             foreach (weak Meta.Window window in app.get_windows ()) {
                 if (!is_eligible_window (window)) {
@@ -116,7 +112,6 @@ public class Gala.DesktopIntegration : GLib.Object {
                 properties.insert ("client-type", new GLib.Variant.uint32 (window.get_client_type ()));
                 properties.insert ("is-hidden", new GLib.Variant.boolean (window.is_hidden ()));
                 properties.insert ("has-focus", new GLib.Variant.boolean (window.has_focus ()));
-                properties.insert ("on-active-workspace", new GLib.Variant.boolean (window.located_on_workspace (active_workspace)));
                 properties.insert ("workspace-index", new GLib.Variant.int32 (window.get_workspace ().index ()));
                 properties.insert ("width", new GLib.Variant.uint32 (frame_rect.width));
                 properties.insert ("height", new GLib.Variant.uint32 (frame_rect.height));

src/NotificationsManager.vala

@@ -11,14 +11,17 @@ public class Gala.NotificationsManager : GLib.Object {
     [DBus (name = "org.freedesktop.Notifications")]
     private interface DBusNotifications : GLib.Object {
         public signal void action_invoked (uint32 id, string action_key);
+        public signal void notification_closed (uint32 id, uint32 reason);
 
         public abstract async uint32 notify (string app_name, uint32 replaces_id, string app_icon, string summary,
             string body, string[] actions, HashTable<string, Variant> hints, int32 expire_timeout) throws DBusError, IOError;
     }
 
     private const int EXPIRE_TIMEOUT = 2000;
 
-    private GLib.SimpleActionGroup action_group = new GLib.SimpleActionGroup ();
+    public signal void action_invoked (uint32 id, string name, GLib.Variant? target_value);
+    public signal void notification_closed (uint32 id);
+
     private DBusNotifications? notifications = null;
     private GLib.HashTable<string, uint32> replaces_id_table = new GLib.HashTable<string, uint32> (str_hash, str_equal);
 
@@ -33,6 +36,7 @@ public class Gala.NotificationsManager : GLib.Object {
                 try {
                     notifications = ((DBusConnection) obj).get_proxy.end<DBusNotifications> (res);
                     notifications.action_invoked.connect (handle_action_invoked);
+                    notifications.notification_closed.connect (handle_notification_closed);
                 } catch (Error e) {
                     warning ("NotificationsManager: Couldn't connect to notifications server: %s", e.message);
                     notifications = null;
@@ -57,16 +61,14 @@ public class Gala.NotificationsManager : GLib.Object {
             return;
         }
 
-        if (action_group.has_action (name)) {
-            action_group.activate_action (name, target_value);
-        }
+        action_invoked (id, name, target_value);
     }
 
-    public void add_action (GLib.Action action) {
-        action_group.add_action (action);
+    private void handle_notification_closed (uint32 id, uint32 reason) {
+        notification_closed (id);
     }
 
-    public async void send (
+    public async uint32? send (
         string component_name,
         string icon,
         string summary,
@@ -76,7 +78,7 @@ public class Gala.NotificationsManager : GLib.Object {
     ) {
         if (notifications == null) {
             warning ("NotificationsManager: Unable to send notification. No connection to notification server");
-            return;
+            return null;
         }
 
         uint32? replaces_id = replaces_id_table.get (component_name);
@@ -97,8 +99,11 @@ public class Gala.NotificationsManager : GLib.Object {
             );
 
             replaces_id_table.insert (component_name, notification_id);
+
+            return notification_id;
         } catch (Error e) {
             critical ("NotificationsManager: There was an error sending a notification: %s", e.message);
+            return null;
         }
     }
 }