From 8d693a7d0e9efd488a233ef8efe91f3355b93885 Mon Sep 17 00:00:00 2001
From: Robin Scholtes <robin@theweirdscience.com>
Date: Wed, 26 Apr 2023 23:24:30 +1200
Subject: [PATCH] [#181] tidying up with docs and notes for tomorrow :)

---
 src/web/App.mjs | 26 ++++++++++++++++++++++----
 1 file changed, 22 insertions(+), 4 deletions(-)

diff --git a/src/web/App.mjs b/src/web/App.mjs
index 9d286b29..755216a7 100755
--- a/src/web/App.mjs
+++ b/src/web/App.mjs
@@ -330,8 +330,6 @@ class App {
         });
 
         this.adjustComponentSizes();
-
-        // @TODO: handle sortable, draggable, popovers functionality etc.
     }
 
     /**
@@ -882,16 +880,36 @@ class App {
         }
     }
 
+    /**
+     * A collection of function calls that need to fire on
+     * window resizing when the window inner width >= the
+     * breakpoint
+     *
+     * @param {boolean} minimise
+     */
     setDesktopUI(minimise){
         this.setDesktopLayout(minimise);
-        // we don't want to display any checkmarks on desktop, so we clear them
+
+        /**
+         * We don't want to display any checkmarks on desktop, so we clear them.
+         * It has no effect on the recipe list, it's purely a visual indicator
+         */
         this.manager.recipe.clearAllSelectedClasses();
+
+        // @TODO: handle sortable, draggable, popovers functionality. Works fine on init but needs to work on window resizing too
     }
 
+    /**
+     * A collection of function calls that need to fire on
+     * window resizing when the window inner width < the
+     * breakpoint
+     */
     setMobileUI(){
         this.setMobileLayout();
-        // on window resizing below breakpoint, we want to put the checkmarks back
+        // on window resizing below breakpoint, we need to put the checkmarks back
         this.manager.recipe.updateSelectedOperations();
+
+        // @TODO: handle sortable, draggable, popovers functionality. Works fine on init but needs to work on window resizing too
     }
 }