From 40ea93396d99726bf8fbb31ef4bf83585592e80b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Fri, 6 Aug 2021 18:15:40 +0200 Subject: [PATCH] appDisplay: Avoid a warning At least the last stable release of gjs has an issue with async handlers for custom (i.e. defined in JS) GObject signals. The handler still works, but evoking it prints the following warning: JS ERROR: Error: Could not guess unspecified GValue type We can avoid the warning by using the addAction() convenience method, which makes for a small nice cleanup anyway. https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/4531 --- js/ui/appDisplay.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index f86a7bb67..012e88bd0 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -3531,8 +3531,7 @@ var AppIconMenu = class AppIconMenu extends PopupMenu.PopupMenu { if (Shell.AppSystem.get_default().lookup_app('org.gnome.Software.desktop')) { this._appendSeparator(); - let item = this._appendMenuItem(_("Show Details")); - item.connect('activate', async () => { + this.addAction(_('Show Details'), async () => { let id = this._source.app.get_id(); let args = GLib.Variant.new('(ss)', [id, '']); const bus = await Gio.DBus.get(Gio.BusType.SESSION, null); -- 2.31.1