From f6588caae3bee64e06a9a817187a462f3b50027d Mon Sep 17 00:00:00 2001
From: matt1432 <matt@nelim.org>
Date: Mon, 25 Sep 2023 12:43:36 -0400
Subject: [PATCH] refactor(ags closer): make array of windows not to close to
 clarify code

---
 config/ags/js/misc/closer.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/config/ags/js/misc/closer.js b/config/ags/js/misc/closer.js
index 7337fdd5..9e006271 100644
--- a/config/ags/js/misc/closer.js
+++ b/config/ags/js/misc/closer.js
@@ -1,13 +1,19 @@
 const { Window, EventBox } = ags.Widget;
 const { closeWindow } = ags.App;
 
+const ALWAYS_OPEN = [
+  'closer',
+  'bar',
+  'notifications',
+];
+
 // TODO: close on scroll event too?
 export const closeAll = () => {
   ags.App.windows.forEach(w => {
-    if (w.name != 'bar' &&
-        w.name != 'notifications')
+    if (!ALWAYS_OPEN.some(window => window === w.name))
       ags.App.closeWindow(w.name)
   });
+  closeWindow('closer');
 };
 
 export const Closer = Window({
@@ -18,9 +24,11 @@ export const Closer = Window({
 
   child: EventBox({
     onPrimaryClickRelease: () => closeAll(),
-    connections: [[ags.App, (box, windowName, visible) => {
-      if (!Array.from(ags.App.windows).some(w => w[1].visible && w[0] != 'bar' && w[0] != 'notifications' && w[0] != 'closer')) {
+    connections: [[ags.App, (_b, _w, _v) => {
+      if (!Array.from(ags.App.windows).some(w => w[1].visible &&
+        !ALWAYS_OPEN.some(window => window === w[0]))) {
         closeWindow('closer');
+        print('hi')
       }
     }]],
   }),