From e38c26894be3719fd502199b770985d6182e28a0 Mon Sep 17 00:00:00 2001 From: Mario Sanchez Prada Date: Mon, 8 May 2017 11:38:16 +0100 Subject: [PATCH] popupMenu: Allow specifying an icon on PopupMenuBase.addAction() This allows passing an optional icon parameter to addAction() so that a PopupImageMenuItem instance is created instead of a PopupMenuItem if an icon is specified. https://bugzilla.gnome.org/show_bug.cgi?id=782166 --- js/ui/popupMenu.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js index 53ab7c100..17a5a911a 100644 --- a/js/ui/popupMenu.js +++ b/js/ui/popupMenu.js @@ -467,8 +467,13 @@ const PopupMenuBase = new Lang.Class({ this._setSettingsVisibility(Main.sessionMode.allowSettings); }, - addAction: function(title, callback) { - let menuItem = new PopupMenuItem(title); + addAction: function(title, callback, icon) { + let menuItem; + if (icon != undefined) + menuItem = new PopupImageMenuItem(title, icon); + else + menuItem = new PopupMenuItem(title); + this.addMenuItem(menuItem); menuItem.connect('activate', Lang.bind(this, function (menuItem, event) { callback(event);