From 48c2db1529378697ddfd7ae3324a2444be7d2356 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C5=BEe=20Fortun?= <joze.fortun@gmail.com>
Date: Mon, 2 Oct 2023 16:43:36 +0200
Subject: [PATCH] Improve builder performance with record lists

---
 .../compose/src/components/Common/Grid.vue    | 49 +++++++------------
 .../components/PageBlocks/CalendarBase.vue    |  2 +-
 .../CalendarDisplay.c3.js                     |  1 -
 .../components/PageBlocks/RecordListBase.vue  |  2 +-
 .../PageBlocks/Shared/AutomationTab.c3.js     |  1 -
 .../src/components/PageBlocks/TabsBase.vue    |  2 +-
 .../components/PageBlocks/Wrap/Plain.c3.js    |  1 -
 .../src/components/PageBlocks/base.vue        | 12 ++---
 .../src/components/Public/Page/Grid.vue       |  3 +-
 .../src/views/Admin/Modules/Records/View.vue  |  1 -
 .../compose/src/views/Admin/Pages/Builder.vue |  4 +-
 .../reporter/src/components/Report/Grid.vue   |  4 --
 12 files changed, 31 insertions(+), 51 deletions(-)

diff --git a/client/web/compose/src/components/Common/Grid.vue b/client/web/compose/src/components/Common/Grid.vue
index 3fe27a54da..d353300419 100644
--- a/client/web/compose/src/components/Common/Grid.vue
+++ b/client/web/compose/src/components/Common/Grid.vue
@@ -19,6 +19,7 @@
       :responsive="!editable"
       :use-css-transforms="false"
       class="flex-grow-1 d-flex w-100 h-100"
+      @layout-updated="onLayoutUpdated"
     >
       <template
         v-for="(item, index) in layout"
@@ -37,6 +38,8 @@
           :class="{ 'h-100': isStretchable }"
           class="grid-item"
           style="touch-action: none;"
+          @move="onGridAction"
+          @resize="onGridAction"
           @moved="onBlockUpdated(index)"
           @resized="onBlockUpdated(index)"
         >
@@ -44,7 +47,7 @@
             :block="blocks[item.i]"
             :index="index"
             :block-index="item.i"
-            :bounding-rect="boundingRects[index]"
+            :resizing="resizing"
             v-on="$listeners"
           />
         </grid-item>
@@ -64,8 +67,6 @@
 
 <script>
 import { GridLayout, GridItem } from 'vue-grid-layout'
-import { throttle } from 'lodash'
-import { compose } from '@cortezaproject/corteza-js'
 
 export default {
   i18nOptions: {
@@ -93,8 +94,7 @@ export default {
       // all blocks in vue-grid friendly structure
       layout: [],
 
-      // Grid items bounding rect info
-      boundingRects: [],
+      resizing: false,
     }
   },
 
@@ -137,42 +137,31 @@ export default {
     },
   },
 
-  mounted () {
-    window.addEventListener('resize', this.windowResizeThrottledHandler)
-    this.recalculateBoundingRect()
-  },
-
   beforeDestroy () {
-    this.destroyEvents()
     this.setDefaultValues()
   },
 
   methods: {
-    windowResizeThrottledHandler: throttle(function () { this.recalculateBoundingRect() }, 500),
-
-    // Fetch bounding boxes of all grid items
-    recalculateBoundingRect () {
-      this.boundingRects = (this.$refs.items || [])
-        .map(({ $el }) => {
-          const { x, y, width: w, height: h } = $el.getBoundingClientRect()
-          return { x, y, w, h }
-        })
-    },
-
     onBlockUpdated (index) {
       this.$emit('item-updated', index)
-      this.$emit('update:blocks', this.layout.map(
-        ({ x, y, w, h, i }) => new compose.PageBlockMaker({ ...this.blocks[i], xywh: [x, y, w, h] }),
-      ))
+
+      const { x, y, w, h } = this.layout[index]
+      this.blocks[index].xywh = [x, y, w, h]
     },
 
-    setDefaultValues () {
-      this.layout = []
-      this.boundingRects = []
+    onLayoutUpdated () {
+      this.resizing = false
+    },
+
+    onGridAction () {
+      if (!this.resizing) {
+        this.resizing = true
+      }
     },
 
-    destroyEvents () {
-      window.removeEventListener('resize', this.windowResizeThrottledHandler)
+    setDefaultValues () {
+      this.layout = []
+      this.resizing = false
     },
   },
 }
diff --git a/client/web/compose/src/components/PageBlocks/CalendarBase.vue b/client/web/compose/src/components/PageBlocks/CalendarBase.vue
index af6df135cb..021a12e21f 100644
--- a/client/web/compose/src/components/PageBlocks/CalendarBase.vue
+++ b/client/web/compose/src/components/PageBlocks/CalendarBase.vue
@@ -219,7 +219,7 @@ export default {
       },
     },
 
-    boundingRect: {
+    'block.xywh': {
       deep: true,
       handler () {
         setTimeout(() => {
diff --git a/client/web/compose/src/components/PageBlocks/CalendarConfigurator/CalendarDisplay.c3.js b/client/web/compose/src/components/PageBlocks/CalendarConfigurator/CalendarDisplay.c3.js
index 67a334e68e..f397d27570 100644
--- a/client/web/compose/src/components/PageBlocks/CalendarConfigurator/CalendarDisplay.c3.js
+++ b/client/web/compose/src/components/PageBlocks/CalendarConfigurator/CalendarDisplay.c3.js
@@ -15,7 +15,6 @@ const props = {
     },
   }),
   blockIndex: 7,
-  boundingRect: {},
   mode: 'configurator',
   module: null,
   namespace: new compose.Namespace(),
diff --git a/client/web/compose/src/components/PageBlocks/RecordListBase.vue b/client/web/compose/src/components/PageBlocks/RecordListBase.vue
index a87f4299c3..9c72b7819c 100644
--- a/client/web/compose/src/components/PageBlocks/RecordListBase.vue
+++ b/client/web/compose/src/components/PageBlocks/RecordListBase.vue
@@ -316,7 +316,7 @@
           </b-thead>
 
           <draggable
-            v-if="items.length && !processing"
+            v-if="items.length && !processing && !resizing"
             v-model="items"
             :disabled="!inlineEditing || !options.draggable"
             group="items"
diff --git a/client/web/compose/src/components/PageBlocks/Shared/AutomationTab.c3.js b/client/web/compose/src/components/PageBlocks/Shared/AutomationTab.c3.js
index 8a3464f83f..a52417ed07 100644
--- a/client/web/compose/src/components/PageBlocks/Shared/AutomationTab.c3.js
+++ b/client/web/compose/src/components/PageBlocks/Shared/AutomationTab.c3.js
@@ -18,7 +18,6 @@ const props = {
   namespace: new compose.Namespace(),
   module: new compose.Module(),
   field: new compose.ModuleFieldBool(),
-  boundingRect: {},
   blockIndex: -1,
   page: new compose.Page(),
   block: new compose.PageBlock(),
diff --git a/client/web/compose/src/components/PageBlocks/TabsBase.vue b/client/web/compose/src/components/PageBlocks/TabsBase.vue
index 02a74a9bc3..be02b45085 100644
--- a/client/web/compose/src/components/PageBlocks/TabsBase.vue
+++ b/client/web/compose/src/components/PageBlocks/TabsBase.vue
@@ -44,7 +44,7 @@
       >
         <page-block-tab
           v-if="tab.block"
-          v-bind="{ ...$attrs, ...$props, page, block: tab.block, blockIndex: index, boundingRect: { xywh: block.xywh} }"
+          v-bind="{ ...$attrs, ...$props, page, block: tab.block, blockIndex: index }"
           :record="record"
           :module="module"
           :magnified="magnified"
diff --git a/client/web/compose/src/components/PageBlocks/Wrap/Plain.c3.js b/client/web/compose/src/components/PageBlocks/Wrap/Plain.c3.js
index a58afd1080..68ab257c6e 100644
--- a/client/web/compose/src/components/PageBlocks/Wrap/Plain.c3.js
+++ b/client/web/compose/src/components/PageBlocks/Wrap/Plain.c3.js
@@ -4,7 +4,6 @@ import { compose } from '@cortezaproject/corteza-js'
 
 const props = {
   block: new compose.PageBlock(),
-  boundingRect: {},
 }
 
 export default {
diff --git a/client/web/compose/src/components/PageBlocks/base.vue b/client/web/compose/src/components/PageBlocks/base.vue
index 850d2b44dd..148baa307f 100644
--- a/client/web/compose/src/components/PageBlocks/base.vue
+++ b/client/web/compose/src/components/PageBlocks/base.vue
@@ -12,12 +12,6 @@ export default {
   },
 
   props: {
-    boundingRect: {
-      type: Object,
-      required: false,
-      default: undefined,
-    },
-
     blockIndex: {
       type: Number,
       default: -1,
@@ -67,6 +61,12 @@ export default {
       default: false,
     },
 
+    resizing: {
+      type: Boolean,
+      required: false,
+      default: false,
+    },
+
     magnified: {
       type: Boolean,
       required: false,
diff --git a/client/web/compose/src/components/Public/Page/Grid.vue b/client/web/compose/src/components/Public/Page/Grid.vue
index 96a987e3ba..a54c666a2b 100644
--- a/client/web/compose/src/components/Public/Page/Grid.vue
+++ b/client/web/compose/src/components/Public/Page/Grid.vue
@@ -4,14 +4,13 @@
     :editable="false"
   >
     <template
-      slot-scope="{ boundingRect, block, index }"
+      slot-scope="{ index, block }"
     >
       <page-block
         v-bind="{ ...$attrs }"
         :page="page"
         :blocks="blocks"
         :block="block"
-        :bounding-rect="boundingRect"
         :block-index="index"
         class="p-2"
         v-on="$listeners"
diff --git a/client/web/compose/src/views/Admin/Modules/Records/View.vue b/client/web/compose/src/views/Admin/Modules/Records/View.vue
index c132adea3a..f5559f1944 100644
--- a/client/web/compose/src/views/Admin/Modules/Records/View.vue
+++ b/client/web/compose/src/views/Admin/Modules/Records/View.vue
@@ -144,7 +144,6 @@ export default {
 
       bindParams: {
         page: new compose.Page(),
-        boundingRect: {},
         namespace: this.$attrs.namespace,
       },
 
diff --git a/client/web/compose/src/views/Admin/Pages/Builder.vue b/client/web/compose/src/views/Admin/Pages/Builder.vue
index 02ececd3e1..52e7b5c41d 100644
--- a/client/web/compose/src/views/Admin/Pages/Builder.vue
+++ b/client/web/compose/src/views/Admin/Pages/Builder.vue
@@ -72,7 +72,7 @@
       @item-updated="onBlockUpdated"
     >
       <template
-        slot-scope="{ boundingRect, block, index }"
+        slot-scope="{ index, block, resizing }"
       >
         <div
           :data-test-id="`block-${block.kind}`"
@@ -149,7 +149,7 @@
             :block="block"
             :module="module"
             :record="record"
-            :bounding-rect="boundingRect"
+            :resizing="resizing"
             editable
             class="p-2"
           />
diff --git a/client/web/reporter/src/components/Report/Grid.vue b/client/web/reporter/src/components/Report/Grid.vue
index ec118d96d2..dadfb1ee37 100644
--- a/client/web/reporter/src/components/Report/Grid.vue
+++ b/client/web/reporter/src/components/Report/Grid.vue
@@ -35,7 +35,6 @@
           :block="blocks[item.i]"
           :index="index"
           :block-index="item.i"
-          :bounding-rect="boundingRects[index]"
           v-on="$listeners"
         />
       </grid-item>
@@ -78,9 +77,6 @@ export default {
     return {
       // All blocks in vue-grid friendly structure
       grid: undefined,
-
-      // Grid items bounding rect info
-      boundingRects: [],
     }
   },