show appmenu inf hidden on overview hide

fixes issue #32
hardpixel · Feb 19, 2018 · f42b6eb · f42b6eb
1 parent e71000e
commit f42b6eb
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/[email protected]/modules/appMenu.js b/[email protected]/modules/appMenu.js
@@ -41,6 +41,10 @@ var AppMenu = new Lang.Class({
       'notify::focus-app', Lang.bind(this, this._showMenu)
     );
 
+    this._ovHandlerID = Main.overview.connect(
+      'hiding', Lang.bind(this, this._showMenu)
+    );
+
     this._wmHandlerIDs.push(global.window_manager.connect(
       'destroy', Lang.bind(this, this._updateMenu)
     ));
@@ -75,6 +79,11 @@ var AppMenu = new Lang.Class({
       delete this._wtHandlerID;
     }
 
+    if (this._ovHandlerID) {
+      Main.overview.disconnect(this._ovHandlerID);
+      delete this._ovHandlerID;
+    }
+
     this._wmHandlerIDs.forEach(function (handler) {
       global.window_manager.disconnect(handler);
     });