From e5b81e19f5b4f0335eab12b6a775754eeb049245 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Thu, 30 Jan 2025 12:56:09 +0100 Subject: [PATCH] quickSettings: Hide separator when menu button is hidden Since commit eeddf49371, the menu button in toggles is separated by an actual widget rather than just CSS borders. However the menu button can be hidden, in which case the menu toggle acts as a regular quick toggle. The separator shouldn't be visible either in that case, so hide it. Closes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/8159 Fixes: eeddf49371 ("style: Improve the styles for the separation in quick setting buttons") Part-of: --- js/ui/quickSettings.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/js/ui/quickSettings.js b/js/ui/quickSettings.js index 010c77bd3..c69263904 100644 --- a/js/ui/quickSettings.js +++ b/js/ui/quickSettings.js @@ -189,6 +189,10 @@ export const QuickMenuToggle = GObject.registerClass({ }); this._box.add_child(this._menuButton); + this._menuButton.bind_property('visible', + separator, 'visible', + GObject.BindingFlags.SYNC_CREATE); + this.bind_property('toggle-mode', contents, 'toggle-mode', GObject.BindingFlags.SYNC_CREATE);