bde127b893
This fixes an issue where the indicator can be out of sync until the RfkillManager (used by it) properties change. The problem is that multiple instances of the indicator will use the same RfkillManager instance (getRfkillManager() returns a singleton) that only guarantees to emit the changed signal in two scenarios: when the D-Bus proxy connects and when the proxy properties change. If by the time an indicator is instantiated the RfkillManager's D-Bus proxy is already connected, that indicator would only sync its state when the RfkillManager properties change. Let's fix that by always syncing the state on construction - in the worst case scenario the RfkillManager's D-Bus proxy won't have connected yet and the indicator state will be temporarily out of sync but once it gets connected the indicator will sync again with the correct state. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/1386
113 lines
3.7 KiB
JavaScript
113 lines
3.7 KiB
JavaScript
// -*- mode: js; js-indent-level: 4; indent-tabs-mode: nil -*-
|
|
/* exported Indicator */
|
|
|
|
const { Gio, GObject } = imports.gi;
|
|
const Signals = imports.signals;
|
|
|
|
const Main = imports.ui.main;
|
|
const PanelMenu = imports.ui.panelMenu;
|
|
const PopupMenu = imports.ui.popupMenu;
|
|
|
|
const { loadInterfaceXML } = imports.misc.fileUtils;
|
|
|
|
const BUS_NAME = 'org.gnome.SettingsDaemon.Rfkill';
|
|
const OBJECT_PATH = '/org/gnome/SettingsDaemon/Rfkill';
|
|
|
|
const RfkillManagerInterface = loadInterfaceXML('org.gnome.SettingsDaemon.Rfkill');
|
|
const RfkillManagerProxy = Gio.DBusProxy.makeProxyWrapper(RfkillManagerInterface);
|
|
|
|
var RfkillManager = class {
|
|
constructor() {
|
|
this._proxy = new RfkillManagerProxy(Gio.DBus.session, BUS_NAME, OBJECT_PATH,
|
|
(proxy, error) => {
|
|
if (error) {
|
|
log(error.message);
|
|
return;
|
|
}
|
|
this._proxy.connect('g-properties-changed',
|
|
this._changed.bind(this));
|
|
this._changed();
|
|
});
|
|
}
|
|
|
|
get airplaneMode() {
|
|
return this._proxy.AirplaneMode;
|
|
}
|
|
|
|
set airplaneMode(v) {
|
|
this._proxy.AirplaneMode = v;
|
|
}
|
|
|
|
get hwAirplaneMode() {
|
|
return this._proxy.HardwareAirplaneMode;
|
|
}
|
|
|
|
get shouldShowAirplaneMode() {
|
|
return this._proxy.ShouldShowAirplaneMode;
|
|
}
|
|
|
|
_changed() {
|
|
this.emit('airplane-mode-changed');
|
|
}
|
|
};
|
|
Signals.addSignalMethods(RfkillManager.prototype);
|
|
|
|
var _manager;
|
|
function getRfkillManager() {
|
|
if (_manager != null)
|
|
return _manager;
|
|
|
|
_manager = new RfkillManager();
|
|
return _manager;
|
|
}
|
|
|
|
var Indicator = GObject.registerClass(
|
|
class Indicator extends PanelMenu.SystemIndicator {
|
|
_init() {
|
|
super._init();
|
|
|
|
this._manager = getRfkillManager();
|
|
this._manager.connect('airplane-mode-changed', this._sync.bind(this));
|
|
|
|
this._indicator = this._addIndicator();
|
|
this._indicator.icon_name = 'airplane-mode-symbolic';
|
|
this._indicator.hide();
|
|
|
|
// The menu only appears when airplane mode is on, so just
|
|
// statically build it as if it was on, rather than dynamically
|
|
// changing the menu contents.
|
|
this._item = new PopupMenu.PopupSubMenuMenuItem(_("Airplane Mode On"), true);
|
|
this._item.icon.icon_name = 'airplane-mode-symbolic';
|
|
this._offItem = this._item.menu.addAction(_("Turn Off"), () => {
|
|
this._manager.airplaneMode = false;
|
|
});
|
|
this._item.menu.addSettingsAction(_("Network Settings"), 'gnome-network-panel.desktop');
|
|
this.menu.addMenuItem(this._item);
|
|
|
|
Main.sessionMode.connect('updated', this._sessionUpdated.bind(this));
|
|
this._sessionUpdated();
|
|
|
|
this._sync();
|
|
}
|
|
|
|
_sessionUpdated() {
|
|
let sensitive = !Main.sessionMode.isLocked && !Main.sessionMode.isGreeter;
|
|
this.menu.setSensitive(sensitive);
|
|
}
|
|
|
|
_sync() {
|
|
let airplaneMode = this._manager.airplaneMode;
|
|
let hwAirplaneMode = this._manager.hwAirplaneMode;
|
|
let showAirplaneMode = this._manager.shouldShowAirplaneMode;
|
|
|
|
this._indicator.visible = airplaneMode && showAirplaneMode;
|
|
this._item.visible = airplaneMode && showAirplaneMode;
|
|
this._offItem.setSensitive(!hwAirplaneMode);
|
|
|
|
if (hwAirplaneMode)
|
|
this._offItem.label.text = _("Use hardware switch to turn off");
|
|
else
|
|
this._offItem.label.text = _("Turn Off");
|
|
}
|
|
});
|