From 81eb7db5a0b85c41e6052c3a9fb66ee658798bb0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Wed, 3 Aug 2022 06:03:40 +0200 Subject: [PATCH] status/network: Port to quick settings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit … and just like that, we've reached the end of the journey. There's little else to say, except how remarkably unremarkable the change is (not in terms of UI of course). Part-of: --- js/ui/panel.js | 20 +++---- js/ui/status/network.js | 118 +++++++++++++++++----------------------- 2 files changed, 60 insertions(+), 78 deletions(-) diff --git a/js/ui/panel.js b/js/ui/panel.js index 6337ca0f1..ebe69e450 100644 --- a/js/ui/panel.js +++ b/js/ui/panel.js @@ -372,17 +372,6 @@ class AggregateMenu extends PanelMenu.Button { this._indicators = new St.BoxLayout({ style_class: 'panel-status-indicators-box' }); this.add_child(this._indicators); - - if (Config.HAVE_NETWORKMANAGER) - this._network = new imports.ui.status.network.NMApplet(); - else - this._network = null; - - if (this._network) - this._indicators.add_child(this._network); - - if (this._network) - this.menu.addMenuItem(this._network.menu); } }); @@ -398,6 +387,11 @@ class QuickSettings extends PanelMenu.Button { this.setMenu(new QuickSettingsMenu(this, N_QUICK_SETTINGS_COLUMNS)); + if (Config.HAVE_NETWORKMANAGER) + this._network = new imports.ui.status.network.Indicator(); + else + this._network = null; + if (Config.HAVE_BLUETOOTH) this._bluetooth = new imports.ui.status.bluetooth.Indicator(); else @@ -421,6 +415,8 @@ class QuickSettings extends PanelMenu.Button { this._indicators.add_child(this._thunderbolt); this._indicators.add_child(this._location); this._indicators.add_child(this._nightLight); + if (this._network) + this._indicators.add_child(this._network); this._indicators.add_child(this._darkMode); this._indicators.add_child(this._powerProfiles); if (this._bluetooth) @@ -438,6 +434,8 @@ class QuickSettings extends PanelMenu.Button { this._addItems(this._remoteAccess.quickSettingsItems); this._addItems(this._thunderbolt.quickSettingsItems); this._addItems(this._location.quickSettingsItems); + if (this._network) + this._addItems(this._network.quickSettingsItems); if (this._bluetooth) this._addItems(this._bluetooth.quickSettingsItems); this._addItems(this._powerProfiles.quickSettingsItems); diff --git a/js/ui/status/network.js b/js/ui/status/network.js index 5a351789e..098727f91 100644 --- a/js/ui/status/network.js +++ b/js/ui/status/network.js @@ -1,14 +1,15 @@ // -*- mode: js; js-indent-level: 4; indent-tabs-mode: nil -*- -/* exported NMApplet */ +/* exported Indicator */ const {Atk, Clutter, Gio, GLib, GObject, NM, Polkit, St} = imports.gi; const Main = imports.ui.main; -const PanelMenu = imports.ui.panelMenu; const PopupMenu = imports.ui.popupMenu; const MessageTray = imports.ui.messageTray; const ModemManager = imports.misc.modemManager; const Util = imports.misc.util; +const {QuickMenuToggle, SystemIndicator} = imports.ui.quickSettings; + const {loadInterfaceXML} = imports.misc.fileUtils; const {registerDestroyableType} = imports.misc.signalTracker; @@ -1298,27 +1299,14 @@ const NMVpnConnectionItem = GObject.registerClass({ } }); -const NMSection = GObject.registerClass({ - Properties: { - 'checked': GObject.ParamSpec.boolean('checked', '', '', - GObject.ParamFlags.READWRITE, - false), - 'icon-name': GObject.ParamSpec.string('icon-name', '', '', - GObject.ParamFlags.READWRITE, - ''), - 'label': GObject.ParamSpec.string('label', '', '', - GObject.ParamFlags.READWRITE, - ''), - }, +const NMToggle = GObject.registerClass({ Signals: { 'activation-failed': {}, }, -}, class NMSection extends GObject.Object { +}, class NMToggle extends QuickMenuToggle { constructor() { super(); - this.menu = new PopupMenu.PopupMenuSection(); - this._items = new Map(); this._itemSorter = new ItemSorter({trackMru: true}); @@ -1334,6 +1322,8 @@ const NMSection = GObject.registerClass({ this, 'label', GObject.BindingFlags.DEFAULT, (bind, source) => [true, this._transformLabel(source)], null); + + this.connect('clicked', () => this.activate()); } setClient(client) { @@ -1354,10 +1344,6 @@ const NMSection = GObject.registerClass({ this._sync(); } - set visible(visible) { - this.menu.actor.visible = visible; - } - activate() { const activeItems = [...this._getActiveItems()]; @@ -1472,11 +1458,12 @@ const NMSection = GObject.registerClass({ } }); -const NMVpnSection = GObject.registerClass( -class NMVpnSection extends NMSection { +const NMVpnToggle = GObject.registerClass( +class NMVpnToggle extends NMToggle { constructor() { super(); + this.menu.setHeader('network-vpn-symbolic', _('VPN')); this.menu.addSettingsAction(_('VPN Settings'), 'gnome-network-panel.desktop'); } @@ -1567,8 +1554,8 @@ class NMVpnSection extends NMSection { } }); -const NMDeviceSection = GObject.registerClass( -class NMDeviceSection extends NMSection { +const NMDeviceToggle = GObject.registerClass( +class NMDeviceToggle extends NMToggle { constructor(deviceType) { super(); @@ -1702,13 +1689,8 @@ class NMDeviceSection extends NMSection { } }); -const NMWirelessSection = GObject.registerClass({ - Properties: { - 'menu-enabled': GObject.ParamSpec.boolean('menu-enabled', '', '', - GObject.ParamFlags.READWRITE, - false), - }, -}, class NMWirelessSection extends NMDeviceSection { +const NMWirelessToggle = GObject.registerClass( +class NMWirelessToggle extends NMDeviceToggle { constructor() { super(NM.DeviceType.WIFI); @@ -1716,6 +1698,7 @@ const NMWirelessSection = GObject.registerClass({ this, 'menu-enabled', GObject.BindingFlags.INVERT_BOOLEAN); + this.menu.setHeader('network-wireless-symbolic', _('Wi–Fi')); this.menu.addSettingsAction(_('All Networks'), 'gnome-wifi-panel.desktop'); } @@ -1763,11 +1746,12 @@ const NMWirelessSection = GObject.registerClass({ } }); -const NMWiredSection = GObject.registerClass( -class NMWiredSection extends NMDeviceSection { +const NMWiredToggle = GObject.registerClass( +class NMWiredToggle extends NMDeviceToggle { constructor() { super(NM.DeviceType.ETHERNET); + this.menu.setHeader('network-wired-symbolic', _('Wired Connections')); this.menu.addSettingsAction(_('Wired Settings'), 'gnome-network-panel.desktop'); } @@ -1777,11 +1761,12 @@ class NMWiredSection extends NMDeviceSection { } }); -const NMBluetoothSection = GObject.registerClass( -class NMBluetoothSection extends NMDeviceSection { +const NMBluetoothToggle = GObject.registerClass( +class NMBluetoothToggle extends NMDeviceToggle { constructor() { super(NM.DeviceType.BT); + this.menu.setHeader('network-cellular-symbolic', _('Bluetooth Tethers')); this.menu.addSettingsAction(_('Bluetooth Settings'), 'gnome-network-panel.desktop'); } @@ -1791,11 +1776,13 @@ class NMBluetoothSection extends NMDeviceSection { } }); -const NMModemSection = GObject.registerClass( -class NMModemSection extends NMDeviceSection { +const NMModemToggle = GObject.registerClass( +class NMModemToggle extends NMDeviceToggle { constructor() { super(NM.DeviceType.MODEM); + this.menu.setHeader('network-cellular-symbolic', _('Mobile Connections')); + const settingsLabel = _('Mobile Broadband Settings'); this._wwanSettings = this.menu.addSettingsAction(settingsLabel, 'gnome-wwan-panel.desktop'); @@ -1817,8 +1804,8 @@ class NMModemSection extends NMDeviceSection { } }); -var NMApplet = GObject.registerClass( -class Indicator extends PanelMenu.SystemIndicator { +var Indicator = GObject.registerClass( +class Indicator extends SystemIndicator { _init() { super._init(); @@ -1828,28 +1815,25 @@ class Indicator extends PanelMenu.SystemIndicator { this._notification = null; - this._allSections = []; + this._wiredToggle = new NMWiredToggle(); + this._wirelessToggle = new NMWirelessToggle(); + this._modemToggle = new NMModemToggle(); + this._btToggle = new NMBluetoothToggle(); + this._vpnToggle = new NMVpnToggle(); - this._wiredSection = new NMWiredSection(); - this._wirelessSection = new NMWirelessSection(); - this._modemSection = new NMModemSection(); - this._btSection = new NMBluetoothSection(); - this._vpnSection = new NMVpnSection(); - - this._deviceSections = new Map([ - [NM.DeviceType.ETHERNET, this._wiredSection], - [NM.DeviceType.WIFI, this._wirelessSection], - [NM.DeviceType.MODEM, this._modemSection], - [NM.DeviceType.BT, this._btSection], + this._deviceToggles = new Map([ + [NM.DeviceType.ETHERNET, this._wiredToggle], + [NM.DeviceType.WIFI, this._wirelessToggle], + [NM.DeviceType.MODEM, this._modemToggle], + [NM.DeviceType.BT, this._btToggle], ]); - this._allSections.push(...this._deviceSections.values()); - this._allSections.push(this._vpnSection); + this.quickSettingsItems.push(...this._deviceToggles.values()); + this.quickSettingsItems.push(this._vpnToggle); - this._allSections.forEach(section => { - section.connectObject( + this.quickSettingsItems.forEach(toggle => { + toggle.connectObject( 'activation-failed', () => this._onActivationFailed(), this); - this.menu.addMenuItem(section.menu); }); this._primaryIndicator = this._addIndicator(); @@ -1860,10 +1844,10 @@ class Indicator extends PanelMenu.SystemIndicator { this._primaryIndicator, 'icon-name', GObject.BindingFlags.DEFAULT); - this._vpnSection.bind_property('checked', + this._vpnToggle.bind_property('checked', this._vpnIndicator, 'visible', GObject.BindingFlags.SYNC_CREATE); - this._vpnSection.bind_property('icon-name', + this._vpnToggle.bind_property('icon-name', this._vpnIndicator, 'icon-name', GObject.BindingFlags.SYNC_CREATE); @@ -1873,8 +1857,8 @@ class Indicator extends PanelMenu.SystemIndicator { async _getClient() { this._client = await NM.Client.new_async(null); - this._allSections.forEach( - section => section.setClient(this._client)); + this.quickSettingsItems.forEach( + toggle => toggle.setClient(this._client)); this._client.bind_property('nm-running', this, 'visible', @@ -1891,9 +1875,9 @@ class Indicator extends PanelMenu.SystemIndicator { this._configPermission = await Polkit.Permission.new( 'org.freedesktop.NetworkManager.network-control', null, null); - this._allSections.forEach(section => { + this.quickSettingsItems.forEach(toggle => { this._configPermission.bind_property('allowed', - section, 'reactive', + toggle, 'reactive', GObject.BindingFlags.SYNC_CREATE); }); } catch (e) { @@ -2025,10 +2009,10 @@ class Indicator extends PanelMenu.SystemIndicator { _updateIcon() { const [dev] = this._mainConnection?.get_devices() ?? []; - const primarySection = this._deviceSections.get(dev?.device_type) ?? null; - this._primaryIndicatorBinding.source = primarySection; + const primaryToggle = this._deviceToggles.get(dev?.device_type) ?? null; + this._primaryIndicatorBinding.source = primaryToggle; - if (!primarySection) { + if (!primaryToggle) { if (this._client.connectivity === NM.ConnectivityState.FULL) this._primaryIndicator.icon_name = 'network-wired-symbolic'; else @@ -2037,6 +2021,6 @@ class Indicator extends PanelMenu.SystemIndicator { const state = this._client.get_state(); const connected = state === NM.State.CONNECTED_GLOBAL; - this._primaryIndicator.visible = (primarySection != null) || connected; + this._primaryIndicator.visible = (primaryToggle != null) || connected; } });