refactor

Author: odeimaiz

services/static-webserver/client/source/class/osparc/dashboard/GroupedCardContainer.js

@@ -125,22 +125,27 @@ qx.Class.define("osparc.dashboard.GroupedCardContainer", {
       return control || this.base(arguments, id);
     },
 
+    __modeChanged: function(container) {
+      const spacing = this.getMode() === "grid" ? osparc.dashboard.GridButtonBase.SPACING : osparc.dashboard.ListButtonBase.SPACING;
+      container.getLayout().set({
+        spacingX: spacing,
+        spacingY: spacing
+      });
+      if (this.getMode() === "list") {
+        this.set({
+          expanded: true,
+        });
+      }
+    },
+
     __createContentContainer: function() {
       let contentContainer = null;
       const expanded = this.isExpanded();
       const showAllBtn = this.__expandButton;
       if (expanded) {
         contentContainer = new osparc.dashboard.CardContainer();
-
-        const setContainerSpacing = () => {
-          const spacing = this.getMode() === "grid" ? osparc.dashboard.GridButtonBase.SPACING : osparc.dashboard.ListButtonBase.SPACING;
-          contentContainer.getLayout().set({
-            spacingX: spacing,
-            spacingY: spacing
-          });
-        };
-        setContainerSpacing();
-        this.addListener("changeMode", () => setContainerSpacing());
+        this.__modeChanged(contentContainer);
+        this.addListener("changeMode", () => this.__modeChanged(contentContainer));
         [
           "changeSelection",
           "changeVisibility"

services/static-webserver/client/source/class/osparc/dashboard/ResourceContainerManager.js

@@ -311,17 +311,18 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
       }
     },
 
+    __modeChanged: function(container) {
+      const spacing = this.getMode() === "grid" ? osparc.dashboard.GridButtonBase.SPACING : osparc.dashboard.ListButtonBase.SPACING;
+      container.getLayout().set({
+        spacingX: spacing,
+        spacingY: spacing
+      });
+    },
+
     __createFlatList: function() {
       const flatList = new osparc.dashboard.CardContainer();
-      const setContainerSpacing = () => {
-        const spacing = this.getMode() === "grid" ? osparc.dashboard.GridButtonBase.SPACING : osparc.dashboard.ListButtonBase.SPACING;
-        flatList.getLayout().set({
-          spacingX: spacing,
-          spacingY: spacing
-        });
-      };
-      setContainerSpacing();
-      this.addListener("changeMode", () => setContainerSpacing());
+      this.__modeChanged(flatList);
+      this.addListener("changeMode", () => this.__modeChanged(flatList));
       [
         "changeSelection",
         "changeVisibility"