fix: preserve state of the windows

oae · Apr 13, 2019 · 4715dc1 · 4715dc1
1 parent 878c02c
commit 4715dc1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/appWindow.js b/appWindow.js
@@ -99,7 +99,7 @@ var AppWindow = class AppWindow {
       return;
     }
 
-    if (this.hidden) {
+    if (this.hidden === true) {
       this.show();
     } else {
       this.hide();
@@ -155,7 +155,7 @@ var AppWindow = class AppWindow {
   attach() {
     this.removeCloseButton();
     this.addTray();
-    if (this.hidden) {
+    if (this.hidden === true) {
       this.hide();
     } else {
       this.show();

diff --git a/windowListener.js b/windowListener.js
@@ -98,8 +98,7 @@ var WindowListener = class WindowListener {
       ) {
         const appWindow = new AppWindow(metaWindow);
         const isHidden = currentState.some(
-          ({ pid, idInDec, hidden }) =>
-            metaWindow.get_pid().toString() === pid && hidden === 'true',
+          ({ pid, idInDec, hidden }) => metaWindow.get_pid() === pid && hidden === true,
         );
         if (isHidden) {
           appWindow.hidden = true;
@@ -127,7 +126,10 @@ var WindowListener = class WindowListener {
   _cleanupWindows() {
     let shouldUpdateState = false;
     this.appWindows = this.appWindows.filter(appWin => {
-      if (!windowExists(appWin.pid, appWin.idInDec) || !this.apps.some(app => app.name === appWin.name && app.state === 'enabled')) {
+      if (
+        !windowExists(appWin.pid, appWin.idInDec) ||
+        !this.apps.some(app => app.name === appWin.name && app.state === 'enabled')
+      ) {
         debug(`removing window: ${appWin}`);
         shouldUpdateState = true;
         appWin.destroy();