From b3f29b46cf0bc562d2302c014bde25628802f18b Mon Sep 17 00:00:00 2001 From: Giovanni Campagna Date: Tue, 29 Jan 2013 00:23:31 +0100 Subject: [PATCH] NetworkMenu: defer calling syncSectionTitle until after pushing the device syncSectionTitle looks at device list for the section, to understand if the section should be visible or not, so obviously it needs to see the new device. I wonder when this broke. https://bugzilla.gnome.org/show_bug.cgi?id=692749 --- js/ui/status/network.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/js/ui/status/network.js b/js/ui/status/network.js index ce1e7e44b..8c7c4dc75 100644 --- a/js/ui/status/network.js +++ b/js/ui/status/network.js @@ -1934,10 +1934,10 @@ const NMApplet = new Lang.Class({ section.addMenuItem(wrapper.statusItem); section.addMenuItem(wrapper.section); - this._syncSectionTitle(wrapper.category); - let devices = this._devices[wrapper.category].devices; devices.push(wrapper); + + this._syncSectionTitle(wrapper.category); }, _deviceRemoved: function(client, device) {