From d4582491f540c0faad10d43e0f571eb4508ce173 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Mon, 24 Jun 2019 23:26:41 +0200 Subject: [PATCH] system: Drop custom styling of user submenu icon The default style we get from .popup-menu-icon actually works better, as it ensures consistency with all the other submenu icons used in the aggregate menu. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/591 --- data/theme/gnome-shell-sass/_common.scss | 5 ----- js/ui/status/system.js | 5 +---- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/data/theme/gnome-shell-sass/_common.scss b/data/theme/gnome-shell-sass/_common.scss index 32a6df727..0b38c9c53 100644 --- a/data/theme/gnome-shell-sass/_common.scss +++ b/data/theme/gnome-shell-sass/_common.scss @@ -1155,11 +1155,6 @@ StScrollBar { // a little unstructured mess: - .system-switch-user-submenu-icon { - icon-size: 16px; - padding: 0 4px; - } - #appMenu { spinner-image: url("resource:///org/gnome/shell/theme/process-working.svg"); spacing: 4px; diff --git a/js/ui/status/system.js b/js/ui/status/system.js index a7aa171cf..68c20e224 100644 --- a/js/ui/status/system.js +++ b/js/ui/status/system.js @@ -186,10 +186,7 @@ var Indicator = class extends PanelMenu.SystemIndicator { let item; this._switchUserSubMenu = new PopupMenu.PopupSubMenuMenuItem('', true); - this._switchUserSubMenu.icon.set({ - icon_name: 'avatar-default-symbolic', - style_class: 'system-switch-user-submenu-icon' - }); + this._switchUserSubMenu.icon.icon_name = 'avatar-default-symbolic'; // Since the label of the switch user submenu depends on the width of // the popup menu, and we can't easily connect on allocation-changed